[11] RFR(S) 8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed May 2 22:13:51 UTC 2018


http://cr.openjdk.java.net/~kvn/8202552/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8202552

Stefan K. found several places where #ifdef instead of #if is used for 
INCLUDE_JVMCI.
I also found places where we can use COMPILER2_OR_JVMCI.

An other problem surprised me that we don't set INCLUDE_AOT to 1. 
Makefile defines that variable without value. I changed code to match 
other variables setting - in makefile set it to 0 if it is not part of 
build and set it to 1 in macros.hpp if it is not defined.

Tested with tier1, tier2 and tier2 with Graal as JIT compiler.

-- 
Thanks,
Vladimir



More information about the build-dev mailing list