[PATCH] D88137: [X86] Parse data32 call in .code16 as CALLpcrel32

[Fangrui Song via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rG3d38a975d72c: [X86] Parse data32 call in .code16 as CALLpcrel32 (authored by MaskRay).

Changed prior to commit:
  https://reviews.llvm.org/D88137?vs=293662&id=293914#toc

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88137/new/

https://reviews.llvm.org/D88137

Files:
  llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
  llvm/test/MC/X86/data-prefix-fail.s
  llvm/test/MC/X86/x86-16.s


Index: llvm/test/MC/X86/x86-16.s
===================================================================
--- llvm/test/MC/X86/x86-16.s
+++ llvm/test/MC/X86/x86-16.s
@@ -546,8 +546,12 @@
 jmp	$0x7ace,$0x7ace
 ljmp	$0x7ace,$0x7ace
 
+// CHECK: calll a
+// CHECK: calll a
 // CHECK: calll a
  calll a
+data32 call a
+data32 callw a
 
 // CHECK:	incb	%al # encoding: [0xfe,0xc0]
 	incb %al
Index: llvm/test/MC/X86/data-prefix-fail.s
===================================================================
--- llvm/test/MC/X86/data-prefix-fail.s
+++ llvm/test/MC/X86/data-prefix-fail.s
@@ -23,3 +23,10 @@
 // 32: encoding: [0x0f,0x01,0x15,0x00,0x00,0x00,0x00]
 // ERR16: error: redundant data16 prefix
 data16 lgdt 0
+
+// 64:      data16    # encoding: [0x66]
+// 64-NEXT: callq  0  # encoding: [0xe8,A,A,A,A]
+// 32:      data16    # encoding: [0x66]
+// 32-NEXT: calll  0  # encoding: [0xe8,A,A,A,A]
+// ERR16: {{.*}}.s:[[#@LINE+1]]:1: error: redundant data16 prefix
+data16 call 0
Index: llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
===================================================================
--- llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
+++ llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp
@@ -3012,6 +3012,17 @@
       return Error(NameLoc, "'data32' is not supported in 64-bit mode");
     // Hack to 'data16' for the table lookup.
     PatchedName = "data16";
+
+    if (getLexer().isNot(AsmToken::EndOfStatement)) {
+      StringRef Next = Parser.getTok().getString();
+      // Parse data32 call as calll.
+      if (Next == "call" || Next == "callw") {
+        getLexer().Lex();
+        Name = "calll";
+        PatchedName = Name;
+        isPrefix = false;
+      }
+    }
   }
 
   Operands.push_back(X86Operand::CreateToken(PatchedName, NameLoc));


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88137.293914.patch
Type: text/x-patch
Size: 1766 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200924/18740905/attachment.bin>