procyberian/nasm
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

insns.dat: get rid of the X64 marker (= X86_64,LONG)

Browse source 
The X64 marker for "X86_64,LONG" has turned out to be a problem in
that it is easy to mistake for "long mode" when adding new
instructions, which results in duplicate CPU flags. Kill it off; it
isn't like we will legitimately have new instructions with this
pattern ever again.

Signed-off-by: H. Peter Anvin (Intel) <hpa@zytor.com>
This commit is contained in:
H. Peter Anvin 2020-07-17 19:25:19 -07:00
parent 8ce4a58b1a
commit 6c1ad43d5e
2 changed files with 419 additions and 393 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

804
x86/insns.dat
View file

File diff suppressed because it is too large Load diff

8
x86/insns.pl
View file

@ -502,13 +502,11 @@ sub format_insn($$$$$) {
# expand and uniqify the flags
my %flags;
foreach my $flag (split(',', $flags)) {
next if ($flag eq '');
if ($flag eq 'ND') {
$nd = 1;
} elsif ($flag eq 'X64') {
# X64 is shorthand for "LONG,X86_64"
$flags{'LONG'}++;
$flags{'X86_64'}++;
} elsif ($flag ne '') {
} else {
$flags{$flag}++;
}