VTA RTL patch review?

Richard Henderson rth@redhat.com
Mon Aug 3 19:59:00 GMT 2009


On 07/31/2009 04:32 PM, Richard Henderson wrote:
> On 07/23/2009 12:56 PM, Alexandre Oliva wrote:
>> http://gcc.gnu.org/ml/gcc-patches/2009-07/msg00256.html
> ... and ran out of time in sel-sched.c. To be continued...

There's quite a lot of code in sel-sched.c devoted to
making sure that blocks consisting of only debug insns
don't affect scheduling in ways that a non-debug
compilation wouldn't.

But if that's the case, haven't we already differed
from non-debug compilation?  Surely empty blocks are
removed by cleanup cfg?

Seems to me a better way to handle this is to have
cleanup cfg notice blocks with just debug insns and
move the debug insns somewhere else.  Either into the
predecessor or successor block, depending on what
seems best -- though I'm not sure it should matter
either way.


r~



More information about the Gcc-patches mailing list