RFR: JDK-8077824: Introduce DefineNativeToolchain to handle toolchain configurations

Tim Bell tim.bell at oracle.com
Wed Apr 15 18:48:32 UTC 2015


Erik:

> The macro SetupNativeCompilation handles various overrides of the 
> compiler and linker used for compilation. There is also a LANG option, 
> which implies that it needs to be told if it's compiling C or C++ when 
> in reality, this is handled automatically. The real purpose of the 
> LANG option is just to decide which executable to use for linking in 
> certain conditions/platforms.
>
> I would like to put some more structure around this by introducing a 
> new macro, DefineNativeToolchain. There s a default and a couple of 
> others, extending the default, for linking with the C++ compiler, 
> building for the build platform etc. These definitions will help 
> ensure that all the necessary executables and options are overridden 
> for each of these usecases.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8077824
>
> Webrevs:
>
> The implementation is in NativeCompilation.gmk. I took the liberty of 
> cleaning out some unused compiler/linker variables.
> Top: http://cr.openjdk.java.net/~erikj/8077824/webrev.root.01/
>
> The jdk changes are mostly just removing "LANG:=" and replacing it 
> when necessary with "TOOLCHAIN:=TOOLCHAIN_LINK_CXX". Note that this is 
> not needed for Windows only configurations.
> Jdk: http://cr.openjdk.java.net/~erikj/8077824/webrev.jdk.01/

Nice cleanup.  Looks good to me.

Tim





More information about the build-dev mailing list