On 2019-02-07 10:36, David Holmes wrote:
Hi Matthias,
On 7/02/2019 1:17 am, Baesken, Matthias wrote:
Hello, please review this small change .
I noticed (when looking into AIX xlc16 / xlclang++) the
following: in the case that clang is used for compilation,
HOTSPOT_BUILD_COMPILER claims to be a gcc .
This is a bit misleading.
This change adjusts the setting for clang usage on AIX (for future
usage with xlclang++) and macOSX.
Additionally I removed some old HOTSPOT_BUILD_COMPILER for legacy
Oracle / Sun Studio versions ).
Possibly more could be removed :)
Changes seem fine.
I've always been irked by the fact we can access an actual version
string from the compilers themselves, so we wouldn't need this code.
How do you do that?
I'm having this kind of background low-intensity itch to fix the
duplication between hotspot and build system in this (and related system
detection code). It never rises to enough annoyance to actually get me
to do anything about it, though.
/Magnus
Thanks,
David
Bug/webrev :
https://bugs.openjdk.java.net/browse/JDK-8218562
http://cr.openjdk.java.net/~mbaesken/webrevs/8218562.0/
Thanks, Matthias
