[PATCH] x86: Don't display --32/--64/--x32 without BFD64
H.J. Lu
hjl.tools@gmail.com
Fri Aug 10 18:09:00 GMT 2018
For 32-bit x86 assembler, --64 and --x32 are unsupported if BFD64 is
undefined. Even if BFD64 is defined, --64 and --x32 still may not be
supported if x86-64 support isn't compiled in:
[hjl@gnu-hsw-1 gas]$ ./as-new --64 -o x.o x.s
Assembler messages:
Fatal error: no compiled in support for x86_64
[hjl@gnu-hsw-1 gas]$ ./as-new --x32 -o x.o x.s
Assembler messages:
Fatal error: no compiled in support for 32bit x86_64
[hjl@gnu-hsw-1 gas]$
This patch removes --32/--64/--x32 from md_show_usage if BFD64 is
undefined and runs code64-inval only if BFD64 is undefined.
* config/tc-i386.c (md_show_usage): Don't display --32/--64/--x32
if BFD64 is undefined.
* testsuite/gas/i386/i386.exp (gas_bfd64_check): New.
Run code64-inval if gas_bfd64_check fails.
---
gas/ChangeLog | 7 +++++++
gas/config/tc-i386.c | 4 ++--
gas/testsuite/gas/i386/i386.exp | 9 ++++++++-
3 files changed, 17 insertions(+), 3 deletions(-)
diff --git a/gas/ChangeLog b/gas/ChangeLog
index 2df3fdb4e7..fe86913fc7 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,10 @@
+2018-08-10 H.J. Lu <hongjiu.lu@intel.com>
+
+ * config/tc-i386.c (md_show_usage): Don't display --32/--64/--x32
+ if BFD64 is undefined.
+ * testsuite/gas/i386/i386.exp (gas_bfd64_check): New.
+ Run code64-inval if gas_bfd64_check fails.
+
2018-08-10 H.J. Lu <hongjiu.lu@intel.com>
* testsuite/gas/i386/evex-no-scale.s: Removed.
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index ddbffb95b1..50c5bfa352 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -11127,8 +11127,8 @@ md_show_usage (FILE *stream)
fprintf (stream, _("\
-s ignored\n"));
#endif
-#if (defined (OBJ_ELF) || defined (OBJ_MAYBE_ELF) \
- || defined (TE_PE) || defined (TE_PEP))
+#if defined BFD64 && (defined (OBJ_ELF) || defined (OBJ_MAYBE_ELF) \
+ || defined (TE_PE) || defined (TE_PEP))
fprintf (stream, _("\
--32/--64/--x32 generate 32bit/64bit/x32 code\n"));
#endif
diff --git a/gas/testsuite/gas/i386/i386.exp b/gas/testsuite/gas/i386/i386.exp
index 6ea4deef34..aa1a0f298d 100644
--- a/gas/testsuite/gas/i386/i386.exp
+++ b/gas/testsuite/gas/i386/i386.exp
@@ -33,6 +33,13 @@ proc gas_32_check { } {
return [regexp "targets:.*i386" [lindex $status 1]];
}
+proc gas_bfd64_check { } {
+ global AS
+
+ set status [gas_host_run "$AS --help" ""]
+ return [regexp "32bit/64bit/x32" [lindex $status 1]];
+}
+
if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_32_check]] then {
global ASFLAGS
@@ -534,7 +541,7 @@ if [expr ([istarget "i*86-*-*"] || [istarget "x86_64-*-*"]) && [gas_32_check]]
run_dump_test "intel-regs"
run_dump_test "mixed-mode-reloc32"
run_dump_test "code64"
- } else {
+ } elseif { ![gas_bfd64_check] } {
run_list_test "code64-inval" "-I${srcdir}/$subdir -al"
}
--
2.17.1
More information about the Binutils
mailing list