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