https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71991

--- Comment #6 from Martin Liška <marxin at gcc dot gnu.org> ---
(In reply to Jan Hubicka from comment #5)
> Created attachment 43798 [details]
> proposed fix
> 
> 
> this patch simply while-lists some transitions of target flags for always
> inline functions. It is ugly but I can't think of anything else which would
> look safe.
> Martin, you mentioned there was packages broken by this. Perhaps we can try
> rebuild with this patch to see if all of the real world issues are solved?
> If not, i guess we will need to decide case-by-case what is safe and what
> not :(

Thanks Honza for the patch. I see 2 failing packages:

1) xf86-video, it uses https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71991#c0
Thus it will be fixed.

2) xen

It contains:

cat fuzz-emul.i
__attribute__((__always_inline__)) a() {}
#pragma GCC target "no-sse"
b() { a(); }

Where 'a' is 'memcpy' and 'b' is a function in xen.

Can you Honza also take a look r251333 where we started to reject such code?

Reply via email to