hg: jdk9/hs-comp/hotspot: 8132168: Support IdealGraphVisualizer in optimized build

Vladimir Ivanov vladimir.x.ivanov at oracle.com
Wed Oct 21 11:08:36 UTC 2015


Some background: there are 2 bugs in play here - 8132168 [1] and 8065151 
[2].

PrintIdealGraphLevel range fix was tracked by 8132168 initially [3], but 
I erroneously used 8065151 in the changeset.

In order to fix that I swapped the contents of the bugs. So, changesets 
and jbs agree on the changes now.

At the same time I noticed that I haven't pushed ex-8065151 yet though 
it was reviewed long ago [4]. So, I finally pushed it.

Sorry for the confusion.

Best regards,
Vladimir Ivanov

[1] https://bugs.openjdk.java.net/browse/JDK-8132168
[2] https://bugs.openjdk.java.net/browse/JDK-8065151
[3] 
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2015-July/018514.html
[4] 
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2014-November/016307.html

On 10/21/15 3:57 AM, Vladimir Kozlov wrote:
> Hi Vladimir
>
> I did not see updated webrev for these changes. The old one was only to
> add range:
>
> http://cr.openjdk.java.net/~vlivanov/8132168/webrev.00/
>
> Changes are fine but we have to follow review process.
>
> Thanks,
> Vladimir K
>
> On 10/21/15 6:47 AM, vladimir.x.ivanov at oracle.com wrote:
>> Changeset: 1cd251540653
>> Author:    vlivanov
>> Date:      2015-10-20 19:22 +0300
>> URL:
>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/1cd251540653
>>
>> 8132168: Support IdealGraphVisualizer in optimized build
>> Reviewed-by: kvn
>>
>> ! src/share/vm/opto/c2_globals.hpp
>>
>> Changeset: 03fa0a35a468
>> Author:    vlivanov
>> Date:      2015-10-20 22:03 +0000
>> URL:
>> http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/rev/03fa0a35a468
>>
>> Merge
>>
>> ! src/share/vm/opto/c2_globals.hpp
>>


More information about the hotspot-compiler-dev mailing list