[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Bug gas/29525] x86: CMPSD and MOVSD wrongly accepted in AT&T mode
From: |
cvs-commit at gcc dot gnu.org |
Subject: |
[Bug gas/29525] x86: CMPSD and MOVSD wrongly accepted in AT&T mode |
Date: |
Mon, 12 Dec 2022 12:54:10 +0000 |
https://sourceware.org/bugzilla/show_bug.cgi?id=29525
--- Comment #3 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot
gnu.org> ---
The master branch has been updated by Jan Beulich <jbeulich@sourceware.org>:
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=04784e33fabb45c4de7a901587f468d4bc169649
commit 04784e33fabb45c4de7a901587f468d4bc169649
Author: Jan Beulich <jbeulich@suse.com>
Date: Mon Dec 12 13:51:46 2022 +0100
x86: re-work insn/suffix recognition
Having templates with a suffix explicitly present has always been
quirky. Introduce a 2nd matching pass in case the 1st one couldn't find
a suitable template _and_ didn't itself already need to trim off a
suffix to find a match at all. This requires error reporting adjustments
(albeit luckily fewer than I was afraid might be necessary), as errors
previously reported during matching now need deferring until after the
2nd pass (because, obviously, we must not emit any error if the 2nd pass
succeeds). While also related to PR gas/29524, it was requested that
move-with-sign-extend be left as broken as it always was.
PR gas/29525
Note that with the dropped CMPSD and MOVSD Intel Syntax string insn
templates taking operands, mixed IsString/non-IsString template groups
(with memory operands) cannot occur anymore. With that
maybe_adjust_templates() becomes unnecessary (and is hence being
removed).
PR gas/29526
Note further that while the additions to the intel16 testcase aren't
really proper Intel syntax, we've been permitting all of those except
for the MOVD variant. The test therefore is to avoid re-introducing such
an inconsistency.
--
You are receiving this mail because:
You are on the CC list for the bug.