Don't include PROP_AUTOINC in PROP_POSTRELOAD

Richard Earnshaw rearnsha@gcc.gnu.org
Mon Dec 13 16:32:00 GMT 2004


On Mon, 2004-12-13 at 16:15, Steven Bosscher wrote:
> Hi,
> 
> This comment in flow.c means we should ever have PROP_AUTOINC after
> reload:
> 
>      (...)
> 
>      We don't want to create new auto-incs after reload, since they
>      are unlikely to be useful and can cause problems with shared
>      stack slots.  */
>   if (reload_completed)
>     flags &= ~(PROP_REG_INFO | PROP_AUTOINC);
> 
> That's interesting, because the definition of PROP_POSTRELOAD does
> include PROP_AUTOINC:
> 
> #define PROP_POSTRELOAD         (PROP_DEATH_NOTES  \
>                                  | PROP_KILL_DEAD_CODE  \
>                                  | PROP_SCAN_DEAD_CODE | PROP_AUTOINC \
>                                  | PROP_SCAN_DEAD_STORES)
> 
> 
> Confusing code, I love it :-)
> 
> I'm going to test the patch below.  OK if it passes?

Grr, no prototype ChangeLog for me to quote :-(

OK.

R.



More information about the Gcc-patches mailing list