aboutsummaryrefslogtreecommitdiff
path: root/gas
diff options
context:
space:
mode:
authorH.J. Lu <hjl.tools@gmail.com>2024-02-05 11:58:08 -0800
committerH.J. Lu <hjl.tools@gmail.com>2024-02-06 03:32:17 -0800
commit0afc614c9938fbf5eda10a26c77d574c3c2f945a (patch)
tree890e0f90dc06442a381769bb5e9821057cff9542 /gas
parentbe908bd6e7fdcf0b91b949b152a063b5791856b5 (diff)
downloadgdb-0afc614c9938fbf5eda10a26c77d574c3c2f945a.zip
gdb-0afc614c9938fbf5eda10a26c77d574c3c2f945a.tar.gz
gdb-0afc614c9938fbf5eda10a26c77d574c3c2f945a.tar.bz2
x86: Warn .insn instruction with length > 15 bytes
Change .insn instruction with length > 15 bytes from error to warning. PR gas/31323 * config/tc-i386.c (output_insn): Issue a warning when .insn instruction length exceeds the limit of 15 bytes. * testsuite/gas/i386/oversized64.s: Add a test for .insn * testsuite/gas/i386/oversized64.l: Updated.
Diffstat (limited to 'gas')
-rw-r--r--gas/config/tc-i386.c10
-rw-r--r--gas/testsuite/gas/i386/oversized64.l8
-rw-r--r--gas/testsuite/gas/i386/oversized64.s3
3 files changed, 19 insertions, 2 deletions
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 8f3a1b6..047aaba 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -11780,8 +11780,14 @@ output_insn (const struct last_insn *last_insn)
{
j = encoding_length (insn_start_frag, insn_start_off, frag_more (0));
if (j > 15)
- as_bad (_("instruction length of %u bytes exceeds the limit of 15"),
- j);
+ {
+ if (dot_insn ())
+ as_warn (_("instruction length of %u bytes exceeds the limit of 15"),
+ j);
+ else
+ as_bad (_("instruction length of %u bytes exceeds the limit of 15"),
+ j);
+ }
else if (fragP)
{
/* NB: Don't add prefix with GOTPC relocation since
diff --git a/gas/testsuite/gas/i386/oversized64.l b/gas/testsuite/gas/i386/oversized64.l
index ac32c4d..f709456 100644
--- a/gas/testsuite/gas/i386/oversized64.l
+++ b/gas/testsuite/gas/i386/oversized64.l
@@ -4,6 +4,7 @@
.*:7: Error: instruction length.*
.*:9: Error: instruction length.*
.*:10: Error: instruction length.*
+.*:13: Warning: instruction length.*
GAS LISTING .*
@@ -37,4 +38,11 @@ GAS LISTING .*
[ ]*10[ ]+48812CC5[ ]*
[ ]*10[ ]+00000000[ ]*
[ ]*10[ ]+44332211[ ]*
+[ ]*11[ ]+
+[ ]*12[ ]+\.att_syntax prefix
+[ ]*13[ ]+\?\?\?\? 6762F1FC[ ]+.insn EVEX.L0.0f 12/0, \$0x11223344,\(,%eax,8\),%rax
+\*\*\*\* Warning: instruction length of 16 bytes exceeds the limit of 15
+[ ]*13[ ]+080C04C5[ ]*
+[ ]*13[ ]+00000000[ ]*
+[ ]*13[ ]+44332211[ ]*
#pass
diff --git a/gas/testsuite/gas/i386/oversized64.s b/gas/testsuite/gas/i386/oversized64.s
index 9db09af..77a08a9 100644
--- a/gas/testsuite/gas/i386/oversized64.s
+++ b/gas/testsuite/gas/i386/oversized64.s
@@ -8,3 +8,6 @@ long64:
xacquire lock add qword ptr gs:[eax*8], 0x11223344
xrelease lock sub qword ptr gs:[eax*8], 0x11223344
+
+ .att_syntax prefix
+ .insn EVEX.L0.0f 12/0, $0x11223344,(,%eax,8),%rax