[11] RFR(S) 8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT
Stefan Karlsson
stefan.karlsson at oracle.com
Thu May 3 07:20:16 UTC 2018
Looks good to me. Thanks for fixing.
StefanK
On 2018-05-03 00:13, Vladimir Kozlov wrote:
> 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.
>
More information about the build-dev
mailing list