review (M) for 6862956: PhaseIdealLoop should have a CFG verification mode

[Tom Rodriguez]

Yes, the indenting seems wrong.  I've fixed it:

Accumulated compiler times:
---------------------------
   Total compilation: 2.887 sec.
     method compilation   : 2.874 sec/50848 bytes (17694 bytes per sec)
     stub compilation     : 0.013 sec.
   Phases:
     parse          : 0.847 sec
     optimizer      : 0.624 sec
       iterGVN        : 0.042 sec
       idealLoop      : 0.284 sec
       idealLoopVerify: 0.101 sec
       ccp            : 0.015 sec
       iterGVN2       : 0.027 sec
       graphReshape   : 0.011 sec
       subtotal       : 0.352 sec,  56.37 %
     matcher        : 0.180 sec
     scheduler      : 0.200 sec
     regalloc       : 0.789 sec
       ctorChaitin    : 0.001 sec
       buildIFG       : 0.272 sec
       computeLive    : 0.172 sec
       regAllocSplit  : 0.141 sec
       postAllocCopyRemoval: 0.044 sec
       fixupSpills    : 0.003 sec
       subtotal       : 0.633 sec,  80.28 %
     macroExpand    : 0.055 sec
     blockOrdering  : 0.018 sec
     peephole       : 0.002 sec
     codeGen        : 0.146 sec
     install_code   : 0.035 sec
     -------------- : ----------
     total          : 2.908 sec,  101.19 %
     output         : 0.147 sec
       isched         : 0.000 sec
       bldOopMaps     : 0.028 sec

tom

On Aug 13, 2009, at 12:12 AM, Christian Thalinger wrote:

> Tom Rodriguez wrote:
>> Accumulated compiler times:
>> ---------------------------
>>   Total compilation: 19.067 sec.
>>     method compilation : 19.057 sec/135529 bytes (7112 bytes per sec)
>>     stub compilation   : 0.009 sec.
>>   Phases:
>>     parse        : 4.822 sec
>>     optimizer    : 2.652 sec
>>       iterGVN      : 0.138 sec
>>       idealLoop    : 1.456 sec
>>     idealLoopVerify: 0.332 sec
>>       ccp          : 0.046 sec
>>       iterGVN2     : 0.164 sec
>>       graphReshape : 0.030 sec
>>       subtotal     : 1.670 sec,  62.95 %
>>     matcher      : 0.525 sec
>>     scheduler    : 0.629 sec
>>     regalloc     : 4.276 sec
>>       ctorChaitin  : 0.002 sec
>>       buildIFG     : 1.425 sec
>>       computeLive  : 0.759 sec
>>       regAllocSplit: 1.260 sec
>>       postAllocCopyRemoval: 0.140 sec
>>       fixupSpills  : 0.012 sec
>>       subtotal     : 3.598 sec,  84.14 %
>>     macroExpand  : 0.140 sec
>>     blockOrdering: 0.049 sec
>>     peephole     : 0.007 sec
>>     codeGen      : 4.347 sec
>>     install_code : 0.057 sec
>>     ------------ : ----------
>>     total        : 17.536 sec,  92.02 %
>>     output    : 4.348 sec
>>       isched    : 0.000 sec
>>       bldOopMaps: 0.088 sec
>
> The indent is a little confusing.  It looks like idealLoopVerify is a
> compiler phase.
>
> -- Christian