The `amdgcn' target is not supported by GAS or LD, so disable building 
in the respective subdirectories, removing configuration errors such as:

This target is no longer supported in gas
make[1]: *** [Makefile:5473: configure-gas] Error 1

and:

*** ld does not support target amdgcn-unknown-none
*** see ld/configure.tgt for supported targets
make[1]: *** [Makefile:6968: configure-ld] Error 1

in builds where no explicit `--disable-gas' and `--disable-ld' options 
have been used with the top-level `configure' script.  Users must not 
have to disable features selected by default to get a working 
configuration.

        /
        * configure.ac <amdgcn-*-*> (noconfigdirs): Add `ld' and `gas'.
        * configure: Regenerate.
---
Hi,

 This is following my observation at 
<https://inbox.sourceware.org/binutils/[email protected]/>,
 
and I guess fallout from changes around binutils commit 978602e83f03 
("bfd: add AMDGCN architecture") having not updated the script.

 OK to apply?  Obviously correct probably.

  Maciej
---
 configure    |    1 +
 configure.ac |    1 +
 2 files changed, 2 insertions(+)

binutils-amdgcn-no-gas-ld.diff
Index: binutils-gdb/configure
===================================================================
--- binutils-gdb.orig/configure
+++ binutils-gdb/configure
@@ -3939,6 +3939,7 @@ case "${target}" in
     noconfigdirs="$noconfigdirs sim target-rda"
     ;;
   amdgcn*-*-*)
+    noconfigdirs="$noconfigdirs ld gas"
     ;;
   arm-*-darwin*)
     noconfigdirs="$noconfigdirs ld gas gdb gprof"
Index: binutils-gdb/configure.ac
===================================================================
--- binutils-gdb.orig/configure.ac
+++ binutils-gdb/configure.ac
@@ -1108,6 +1108,7 @@ case "${target}" in
     noconfigdirs="$noconfigdirs sim target-rda"
     ;;
   amdgcn*-*-*)
+    noconfigdirs="$noconfigdirs ld gas"
     ;;
   arm-*-darwin*)
     noconfigdirs="$noconfigdirs ld gas gdb gprof"

Reply via email to