hg: jdk9/hs-comp/hotspot: 8132168: Support IdealGraphVisualizer in optimized build
Vladimir Kozlov
vladimir.kozlov at oracle.com
Wed Oct 21 13:33:29 UTC 2015
Now it makes sense.
Thanks,
Vladimir
On 10/21/15 7:08 PM, Vladimir Ivanov wrote:
> 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