(XS) RFR: 7008747: Header files with conditional behaviour can not be precompiled
David Holmes
david.holmes at oracle.com
Tue Aug 2 03:29:40 UTC 2016
Bug: https://bugs.openjdk.java.net/browse/JDK-7008747
webrev: http://cr.openjdk.java.net/~dholmes/7008747/webrev/
A trivial clean up to ensure no confusion. The bug might be better
described in the reverse sense: files that modify the behaviour of
included headers must not use precompiled headers.
In this case the code looks like it is using PCH when in fact it will be
disabled on this platform. Better to be 100% clear (like the PPC code
involving the same definitions) and not include precompiled.hpp
If any zero folk see this then zero may also want a cleanup here as it
defines DONT_USE_REGISTER_DEFINES, but doesn't seem to include and
headers that are conditionalized on that value.
Thanks,
David
More information about the hotspot-runtime-dev
mailing list