Re: [PATCH] Fix fallout of my fold_binary EQ/NE patch (PR middle-end/48136)

2011-03-16 Thread Richard Guenther
On Tue, Mar 15, 2011 at 8:41 PM, Jakub Jelinek wrote: > Hi! > > The addition of STRIP_NOPS for EQ_EXPR/NE_EXPR revealed a bunch of cases > which rely on arg0 and arg1 having the same type.  Only the last hunk > fixes the testcase, the rest is what I saw and it was possible the > types wouldn't mat

[PATCH] Fix fallout of my fold_binary EQ/NE patch (PR middle-end/48136)

2011-03-15 Thread Jakub Jelinek
Hi! The addition of STRIP_NOPS for EQ_EXPR/NE_EXPR revealed a bunch of cases which rely on arg0 and arg1 having the same type. Only the last hunk fixes the testcase, the rest is what I saw and it was possible the types wouldn't match. Tried to construct testcases for the other cases, but didn't