[PATCH] D24957: [x86][ms-inline-asm] use of "jmp short" in asm is not supported

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 5 10:10:38 PDT 2016


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



> X86AsmParser.cpp:2142
> +      Parser.Lex();
> +      // MS ignores the short keyword, it determines the jmp type based
> +      // on the distance of the label

I wish we had an explicit "short jump" assembly instruction, though. :( Anyway, for now we should do this for compatibility.

Repository:
  rL LLVM

https://reviews.llvm.org/D24957





More information about the llvm-commits mailing list