Re: [PATCH] Fix expand_cond_expr_using_cmove (PR middle-end/56548)

2013-03-06 Thread Richard Henderson
On 03/06/2013 07:47 AM, Jakub Jelinek wrote: > Hi! > > The http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=193539 > change broke the following testcase on i386. The problem is > that it can return the result in wider mode than callers expect, > causing either ICEs or other issues later on. > >

[PATCH] Fix expand_cond_expr_using_cmove (PR middle-end/56548)

2013-03-06 Thread Jakub Jelinek
Hi! The http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=193539 change broke the following testcase on i386. The problem is that it can return the result in wider mode than callers expect, causing either ICEs or other issues later on. It is fine to perform the cmove in promoted mode, but we shou