[llvm] r309509 - [x86][inline-asm][ms-compat] legalize the use of "jc/jz short <op>"

Coby Tayree via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 30 04:12:47 PDT 2017


Author: coby
Date: Sun Jul 30 04:12:47 2017
New Revision: 309509

URL: http://llvm.org/viewvc/llvm-project?rev=309509&view=rev
Log:
[x86][inline-asm][ms-compat] legalize the use of "jc/jz short <op>"

MS ignores the keyword "short" when used after a jc/jz instruction, LLVM ought to do the same.
Test: D35893

Differential Revision: https://reviews.llvm.org/D35892

Modified:
    llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp

Modified: llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp?rev=309509&r1=309508&r2=309509&view=diff
==============================================================================
--- llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp (original)
+++ llvm/trunk/lib/Target/X86/AsmParser/X86AsmParser.cpp Sun Jul 30 04:12:47 2017
@@ -2309,7 +2309,8 @@ bool X86AsmParser::ParseInstruction(Pars
   InstInfo = &Info;
   StringRef PatchedName = Name;
 
-  if (Name == "jmp" && isParsingIntelSyntax() && isParsingInlineAsm()) {
+  if ((Name.equals("jmp") || Name.equals("jc") || Name.equals("jz")) &&
+      isParsingIntelSyntax() && isParsingInlineAsm()) {
     StringRef NextTok = Parser.getTok().getString();
     if (NextTok == "short") {
       SMLoc NameEndLoc =




More information about the llvm-commits mailing list