Re: Recovering REG_EXPR information after temporary expression replacement

2012-01-27 Thread Peter Bergner
On Fri, 2012-01-27 at 18:40 +0100, Michael Matz wrote: > The hack below works in this specific situation (TERed into a switch), and > adds a REG_EXPR when an TERed SSA name ever expanded into a pseudo (i.e. > also for some more generic situations). FYI, I bootstrapped and regtested your patch on

Re: Recovering REG_EXPR information after temporary expression replacement

2012-01-27 Thread William J. Schmidt
Great, thanks! Results are good on this specific test case. Seems like a nice thing to add in 4.8. On Fri, 2012-01-27 at 18:40 +0100, Michael Matz wrote: > Hi, > > On Fri, 27 Jan 2012, William J. Schmidt wrote: > > > > >

Re: Recovering REG_EXPR information after temporary expression replacement

2012-01-27 Thread Michael Matz
Hi, On Fri, 27 Jan 2012, William J. Schmidt wrote: > > int > test_switch_1 (unsigned int *index, int i) > { > switch (*index) ... > However, for the first case, temporary expression replacement has > effectively removed th

Recovering REG_EXPR information after temporary expression replacement

2012-01-27 Thread William J. Schmidt
A member of our team was working on some switch statement optimizations, and ran into a situation that seems a little curious on the face of it. Consider these two test variants: int test_switch_1 (unsigned int *index, int i)