review (M) for 6862956: PhaseIdealLoop should have a CFG verification mode
Vladimir.Kozlov at Sun.COM
Wed Aug 12 13:59:43 PDT 2009
Tom Rodriguez wrote:
> No I haven't. It's certainly not free. I can add some TracePhases and
> do some runs. I also moved the first verify into the preceeding block
> of PhaseIdealLoop passes so we aren't doing doing multiple verifies for
> code without loops. I'll get some numbers.
I looked more and found that you miss break in the error case in verify_dominance().
> On Aug 12, 2009, at 12:53 PM, Vladimir Kozlov wrote:
>> Thank you, Tom, for doing this.
>> Changes look good.
>> Did you measure how much time compiler spends in the verification code
>> (in PhaseIdealLoop::verify(igvn))?
>> Tom Rodriguez wrote:
More information about the hotspot-compiler-dev