> 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. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/10689/files - new: https://git.openjdk.org/jdk/pull/10689/files/b69fdbaa..8faf5083 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=10689&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10689&range=00-01 Stats: 14194 lines in 298 files changed: 10379 ins; 2288 del; 1527 mod Patch: https://git.openjdk.org/jdk/pull/10689.diff Fetch: git fetch https://git.openjdk.org/jdk pull/10689/head:pull/10689 PR: https://git.openjdk.org/jdk/pull/10689