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

coby via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 26 08:21:06 PDT 2017


coby created this revision.
Herald added a subscriber: eraman.

MS ignores the keyword "short" when used after a jc/jz instruction, LLVM ought to do the same.
llvm: https://reviews.llvm.org/D35892


Repository:
  rL LLVM

https://reviews.llvm.org/D35893

Files:
  test/CodeGen/ms-inline-asm.c


Index: test/CodeGen/ms-inline-asm.c
===================================================================
--- test/CodeGen/ms-inline-asm.c
+++ test/CodeGen/ms-inline-asm.c
@@ -693,10 +693,12 @@
 void label6(){
   __asm {
       jmp short label
+      jc  short label
+      jz  short label
     label:
   }
   // CHECK-LABEL: define void @label6
-  // CHECK: call void asm sideeffect inteldialect "jmp {{.*}}__MSASMLABEL_.${:uid}__label\0A\09{{.*}}__MSASMLABEL_.${:uid}__label:", "~{dirflag},~{fpsr},~{flags}"()
+  // CHECK: jmp {{.*}}__MSASMLABEL_.${:uid}__label\0A\09jc {{.*}}__MSASMLABEL_.${:uid}__label\0A\09jz {{.*}}__MSASMLABEL_.${:uid}__label\0A\09{{.*}}__MSASMLABEL_.${:uid}__label:"
 }
 
 // Don't include mxcsr in the clobber list.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35893.108286.patch
Type: text/x-patch
Size: 740 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170726/31e1fc93/attachment.bin>


More information about the llvm-commits mailing list