Re: Fix PR 63566 part 3

2015-02-09 Thread Jason Merrill
On 02/09/2015 04:01 PM, Jan Hubicka wrote: I had to dro # in the error message, I am not quite sure what it means. It just means more verbose. Jason, what do you think? Looks fine to me. Jason

Re: Fix PR 63566 part 3

2015-02-09 Thread Jan Hubicka
> Jan, > This patch caused Bug 64982 - [5 Regression] Many g++ failures on > x86_64-apple-darwin14 with -m32. >Jack Hi, I think best way would be to move the warning to middle-end when the thunk is being expanded (other alternative is just to force analysis before doing t

Re: Fix PR 63566 part 3

2015-02-09 Thread Jack Howarth
Jan, This patch caused Bug 64982 - [5 Regression] Many g++ failures on x86_64-apple-darwin14 with -m32. Jack On Sun, Feb 8, 2015 at 3:20 PM, Jan Hubicka wrote: > Hi, > this patch makes i386.c ready for presence of aliases of local functions, but > also fixes a wrong code i

Fix PR 63566 part 3

2015-02-08 Thread Jan Hubicka
Hi, this patch makes i386.c ready for presence of aliases of local functions, but also fixes a wrong code issue where ix86_function_sseregparm does use caller TARET_SSE_MATH flags instead of callees. If callee disagree on this flag, it may get called with wrong calling convention. I added error m