> 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

Reply via email to