RFR: 8295262: Build binutils out of source tree [v2]

[Aleksey Shipilev]

On Thu, 13 Oct 2022 08:41:35 GMT, Ludovic Henry <luhenry at openjdk.org> wrote:

>> Currently, when passing --with-binutils-src, binutils is built in the source tree. That leads to conflicting targets when compiling for different architectures (ex: amd64 on the host, and riscv64 or aarch64 for the target) from the same jdk source tree.
>> 
>> The simplest solution is to build binutils out-of-tree and into the build/<target>/binutils folder. These out-of-tree builds are already supported by binutils and only require some changes in the way we invoke the binutils/configure command.
>
> Ludovic Henry has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Remove unrelated change
>  - Merge branch 'master' of github.com:openjdk/jdk into dev/ludovic/upstream-hsdis-cross-compile
>  - 8295262: Build binutils out of source tree
>    
>    Currently, when passing --with-binutils-src, binutils is built in the source tree. That leads to conflicting targets when compiling for different architectures (ex: amd64 on the host, and riscv64 or aarch64 for the target) from the same jdk source tree.
>    
>    The simplest solution is to build binutils out-of-tree and into the build/<target>/binutils folder. These out-of-tree builds are already supported by binutils and only require some changes in the way we invoke the binutils/configure command.

This looks good to me. I tested hsdis builds on these platforms and they pass:


server-release-aarch64-linux-gnu
server-release-arm-linux-gnueabihf
server-release-i686-linux-gnu
server-release-powerpc64le-linux-gnu
server-release-powerpc64-linux-gnu-
server-release-riscv64-linux-gnu
server-release-s390x-linux-gnu
server-release-x86_64-linux-gnu
zero-release-alpha-linux-gnu
zero-release-arm-linux-gnueabi
zero-release-arm-linux-gnueabihf
zero-release-m68k-linux-gnu
zero-release-mips64el-linux-gnuabi64
zero-release-mipsel-linux-gnu
zero-release-powerpc-linux-gnu
zero-release-sh4-linux-gnu
zero-release-sparc64-linux-gnu

-------------

Marked as reviewed by shade (Reviewer).

PR: https://git.openjdk.org/jdk/pull/10689