Thank you, Stefan
Vladimir K
On 5/3/18 12:20 AM, Stefan Karlsson wrote:
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.