RFR: 8156942: [JVMCI] replace LIRKind with abstract base class

Igor Veresov igor.veresov at oracle.com
Tue May 17 17:58:44 UTC 2016


Sill alright.

igor

> On May 17, 2016, at 3:04 AM, Roland Schatz <roland.schatz at oracle.com> wrote:
> 
> Thanks for the review!
> 
> One minor update:
> http://cr.openjdk.java.net/~rschatz/JDK-8156942/webrev.01/
> 
> I missed a reference to the removed LIRKind class inside an assertion, breaking the debug builds.
> 
> - Roland
> 
> On 05/13/2016 06:22 PM, Igor Veresov wrote:
>> Looks ok
>> 
>> igor
>> 
>>> On May 13, 2016, at 8:32 AM, Roland Schatz <roland.schatz at oracle.com> wrote:
>>> 
>>> Hi,
>>> 
>>> Please review this cleanup change:
>>> http://cr.openjdk.java.net/~rschatz/JDK-8156942/webrev.00/
>>> https://bugs.openjdk.java.net/browse/JDK-8156942
>>> 
>>> Passes all jvmci jtreg tests and all graal unit tests.
>>> 
>>> Thanks,
>>> Roland
> 



More information about the hotspot-compiler-dev mailing list