diff mbox series

configure: Adjust several assembler checks to remove an unused parm.

Message ID F27996B8-1183-44C2-8561-279A682E5864@sandoe.co.uk
State New
Headers show
Series configure: Adjust several assembler checks to remove an unused parm. | expand

Commit Message

Iain Sandoe Aug. 26, 2021, 10:44 a.m. UTC
Hi,

noticed while trying to apply a new change to gcc/configure.ac

-------

In r12-3048-ge0b6d0b39c6, the GAS version parameter was removed from
the gcc_GAS_CHECK_FEATURE macro.  It seems that overlapping comit/test
cycles resulted in several AMDGCN and one Darwin commit with the now
extra parameter still present.

This causes wrong configure code to be generated when autoreconf is
used in the gcc directory.

Fixed by removing the extraneous parm from the AMDGCN and Darwin cases.

Checked that this produces “no change” to the configure output when
“autoreconf -I -I../config -I..” is used in the gcc directory.

OK if regstraps succeed on x86_64-linux and darwin?
thanks
Iain

Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>

gcc/ChangeLog:

	* configure.ac (darwin2[[0-9]]* | darwin19*): Alter use of
	gcc_GAS_CHECK_FEATURE to remove an extraneous parameter.
	(amdgcn-* | gcn-*) Likewise.
---
 gcc/configure.ac | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

Richard Biener Aug. 26, 2021, 11:05 a.m. UTC | #1
On Thu, Aug 26, 2021 at 12:44 PM Iain Sandoe <iain@sandoe.co.uk> wrote:
>
> Hi,
>
> noticed while trying to apply a new change to gcc/configure.ac
>
> -------
>
> In r12-3048-ge0b6d0b39c6, the GAS version parameter was removed from
> the gcc_GAS_CHECK_FEATURE macro.  It seems that overlapping comit/test
> cycles resulted in several AMDGCN and one Darwin commit with the now
> extra parameter still present.
>
> This causes wrong configure code to be generated when autoreconf is
> used in the gcc directory.
>
> Fixed by removing the extraneous parm from the AMDGCN and Darwin cases.
>
> Checked that this produces “no change” to the configure output when
> “autoreconf -I -I../config -I..” is used in the gcc directory.
>
> OK if regstraps succeed on x86_64-linux and darwin?

OK

> thanks
> Iain
>
> Signed-off-by: Iain Sandoe <iain@sandoe.co.uk>
>
> gcc/ChangeLog:
>
>         * configure.ac (darwin2[[0-9]]* | darwin19*): Alter use of
>         gcc_GAS_CHECK_FEATURE to remove an extraneous parameter.
>         (amdgcn-* | gcn-*) Likewise.
> ---
>  gcc/configure.ac | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/gcc/configure.ac b/gcc/configure.ac
> index c651db7457f..6f768e02aa4 100644
> --- a/gcc/configure.ac
> +++ b/gcc/configure.ac
> @@ -4709,7 +4709,7 @@ foo:      nop
>      case $target_os in
>         darwin2[[0-9]]* | darwin19*)
>          gcc_GAS_CHECK_FEATURE([llvm assembler x86-pad-for-align option],
> -          gcc_cv_as_mllvm_x86_pad_for_align,,
> +          gcc_cv_as_mllvm_x86_pad_for_align,
>            [-mllvm -x86-pad-for-align=false], [.text],,
>            [AC_DEFINE(HAVE_AS_MLLVM_X86_PAD_FOR_ALIGN, 1,
>             [Define if your Mac OS X assembler supports -mllvm -x86-pad-for-align=false.])])
> @@ -5350,25 +5350,25 @@ esac
>  case "$target" in
>    amdgcn-* | gcn-*)
>      gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for fiji],
> -      gcc_cv_as_gcn_sram_ecc_fiji,,
> +      gcc_cv_as_gcn_sram_ecc_fiji,
>        [-triple=amdgcn--amdhsa -mcpu=fiji -mattr=-xnack -mattr=+sram-ecc],
>        [.amdgcn_target "amdgcn-unknown-amdhsa--gfx803+sram-ecc"],,
>        [AC_DEFINE(HAVE_GCN_SRAM_ECC_FIJI, 1,
>         [Define if your assembler allows -mattr=+sram-ecc for fiji.])])
>      gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx900],
> -      gcc_cv_as_gcn_sram_ecc_gfx900,,
> +      gcc_cv_as_gcn_sram_ecc_gfx900,
>        [-triple=amdgcn--amdhsa -mcpu=gfx900 -mattr=-xnack -mattr=+sram-ecc],
>        [.amdgcn_target "amdgcn-unknown-amdhsa--gfx900+sram-ecc"],,
>        [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX900, 1,
>         [Define if your assembler allows -mattr=+sram-ecc for gfx900.])])
>      gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx906],
> -      gcc_cv_as_gcn_sram_ecc_gfx906,,
> +      gcc_cv_as_gcn_sram_ecc_gfx906,
>        [-triple=amdgcn--amdhsa -mcpu=gfx906 -mattr=-xnack -mattr=+sram-ecc],
>        [.amdgcn_target "amdgcn-unknown-amdhsa--gfx906+sram-ecc"],,
>        [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX906, 1,
>         [Define if your assembler allows -mattr=+sram-ecc for gfx906.])])
>      gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx908],
> -      gcc_cv_as_gcn_sram_ecc_gfx908,,
> +      gcc_cv_as_gcn_sram_ecc_gfx908,
>        [-triple=amdgcn--amdhsa -mcpu=gfx908 -mattr=-xnack -mattr=+sram-ecc],
>        [.amdgcn_target "amdgcn-unknown-amdhsa--gfx908+sram-ecc"],,
>        [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX908, 1,
> --
> 2.24.3 (Apple Git-128)
>
diff mbox series

Patch

diff --git a/gcc/configure.ac b/gcc/configure.ac
index c651db7457f..6f768e02aa4 100644
--- a/gcc/configure.ac
+++ b/gcc/configure.ac
@@ -4709,7 +4709,7 @@  foo:	nop
     case $target_os in
        darwin2[[0-9]]* | darwin19*)
         gcc_GAS_CHECK_FEATURE([llvm assembler x86-pad-for-align option],
-          gcc_cv_as_mllvm_x86_pad_for_align,,
+          gcc_cv_as_mllvm_x86_pad_for_align,
           [-mllvm -x86-pad-for-align=false], [.text],,
           [AC_DEFINE(HAVE_AS_MLLVM_X86_PAD_FOR_ALIGN, 1,
 	    [Define if your Mac OS X assembler supports -mllvm -x86-pad-for-align=false.])])
@@ -5350,25 +5350,25 @@  esac
 case "$target" in
   amdgcn-* | gcn-*)
     gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for fiji],
-      gcc_cv_as_gcn_sram_ecc_fiji,,
+      gcc_cv_as_gcn_sram_ecc_fiji,
       [-triple=amdgcn--amdhsa -mcpu=fiji -mattr=-xnack -mattr=+sram-ecc],
       [.amdgcn_target "amdgcn-unknown-amdhsa--gfx803+sram-ecc"],,
       [AC_DEFINE(HAVE_GCN_SRAM_ECC_FIJI, 1,
        [Define if your assembler allows -mattr=+sram-ecc for fiji.])])
     gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx900],
-      gcc_cv_as_gcn_sram_ecc_gfx900,,
+      gcc_cv_as_gcn_sram_ecc_gfx900,
       [-triple=amdgcn--amdhsa -mcpu=gfx900 -mattr=-xnack -mattr=+sram-ecc],
       [.amdgcn_target "amdgcn-unknown-amdhsa--gfx900+sram-ecc"],,
       [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX900, 1,
        [Define if your assembler allows -mattr=+sram-ecc for gfx900.])])
     gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx906],
-      gcc_cv_as_gcn_sram_ecc_gfx906,,
+      gcc_cv_as_gcn_sram_ecc_gfx906,
       [-triple=amdgcn--amdhsa -mcpu=gfx906 -mattr=-xnack -mattr=+sram-ecc],
       [.amdgcn_target "amdgcn-unknown-amdhsa--gfx906+sram-ecc"],,
       [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX906, 1,
        [Define if your assembler allows -mattr=+sram-ecc for gfx906.])])
     gcc_GAS_CHECK_FEATURE([assembler accepts -mattr=+sram-ecc for gfx908],
-      gcc_cv_as_gcn_sram_ecc_gfx908,,
+      gcc_cv_as_gcn_sram_ecc_gfx908,
       [-triple=amdgcn--amdhsa -mcpu=gfx908 -mattr=-xnack -mattr=+sram-ecc],
       [.amdgcn_target "amdgcn-unknown-amdhsa--gfx908+sram-ecc"],,
       [AC_DEFINE(HAVE_GCN_SRAM_ECC_GFX908, 1,