Messages in this thread | | | Date | Fri, 09 Jan 2009 22:13:48 -0800 (PST) | Subject | Re: [PATCH] Disable branch profiling macros when sparsed. | From | David Miller <> |
| |
From: Alexey Zaytsev <alexey.zaytsev@gmail.com> Date: Sat, 10 Jan 2009 08:57:28 +0300
> The macros produce lots of unneeded warnings when > recursive if(({ .. if() {..} ..})) {..} and such > are substituted. And there is no point in sparsing > them anyway. This is useful if someone decides to > sparse an allyesconfig kernel. > > Signed-off-by: Alexey Zaytsev <alexey.zaytsev@gmail.com>
If even sparse can't handle these things, it's no surprise how many gcc bogus warning problems we've run into because of this hairy if() macro.
I don't think it's a good precedence to do this. It's giving up on trying to implement the branch tracer in a way that results in a kernel build that produces useful warnings.
| |