http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46893

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2010-12-16 
08:02:21 UTC ---
I agree the simplify-rtx.c change makes sense, but disagree on the cfgexpand.c
side.  In theory the VCE doesn't have to be the outermost expression, it can be
embedded in another handled component or another VCE, etc.
We expand tem to op0 and that only ever considers tem's mode, not mode1 or
whatever.  Thus, if op0 is a CONST_INT and thus has no mode, we should look at
tem's mode, which is where it would get its mode from if it had any.

I'm fine with limiting that just to CONST_INTs if you want.  But it also needs
to be done in both places, not just one.

Reply via email to