[10] RFR: 8152470: Add COMPILER2_OR_JVMCI definition

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[10] RFR: 8152470: Add COMPILER2_OR_JVMCI definition

Jamsheed C m
Hi,

request for review,

jbs: https://bugs.openjdk.java.net/browse/JDK-8152470

webrev: http://cr.openjdk.java.net/~jcm/8152470/webrev.00/

desc: cleanup

#if defined(COMPILER2) || INCLUDE_JVMCI changed to #if COMPILER2_OR_JVMC

Best regards,

Jamsheed

Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR: 8152470: Add COMPILER2_OR_JVMCI definition

Vladimir Kozlov
Looks good.

Thanks,
Vladimir

On 11/3/17 10:36 AM, jamsheed wrote:

> Hi,
>
> request for review,
>
> jbs: https://bugs.openjdk.java.net/browse/JDK-8152470
>
> webrev: http://cr.openjdk.java.net/~jcm/8152470/webrev.00/
>
> desc: cleanup
>
> #if defined(COMPILER2) || INCLUDE_JVMCI changed to #if COMPILER2_OR_JVMC
>
> Best regards,
>
> Jamsheed
>
Reply | Threaded
Open this post in threaded view
|

Re: [10] RFR: 8152470: Add COMPILER2_OR_JVMCI definition

Jamsheed C m
Thank you, Vladimir

Best regards,

Jamsheed


On Monday 06 November 2017 11:56 PM, Vladimir Kozlov wrote:

> Looks good.
>
> Thanks,
> Vladimir
>
> On 11/3/17 10:36 AM, jamsheed wrote:
>> Hi,
>>
>> request for review,
>>
>> jbs: https://bugs.openjdk.java.net/browse/JDK-8152470
>>
>> webrev: http://cr.openjdk.java.net/~jcm/8152470/webrev.00/
>>
>> desc: cleanup
>>
>> #if defined(COMPILER2) || INCLUDE_JVMCI changed to #if COMPILER2_OR_JVMC
>>
>> Best regards,
>>
>> Jamsheed
>>