Re: [PATCH] branch tracer: fix freak link error

From: Arnd Bergmann
Date: Fri Feb 12 2016 - 16:49:50 EST


On Friday 12 February 2016 16:45:03 Nicolas Pitre wrote:
>
> It is certainly a worthwhile workaround if it solves all the known
> cases.
>
> Acked-by: Nicolas Pitre <nico@xxxxxxxxxx>
>
> This trick doesn't work with the test case I produced to demonstrate
> this bug though. So we might be affected again in the future, or maybe
> not if we're lucky.

Yes, I think it's clear that there is still something interesting going
on with the compiler and nested __builtin_compatible_p().

I'm currently creating a couple of gcc bug reports and plan to finally submit
this one as well.

Now I have to check if it does anything to the spurious "maybe-uninitialized"
warnings that I saw creeping in with PROFILE_ALL_BRANCHES. My guess is
that it doesn't fix them (I submitted a patch to turn off those
warnings today when PROFILE_ALL_BRANCHES is set), but I'll try anyway.

Arnd