[PATCH] D34739: [X86][InlineAsm][Ms Compatibility]Prefer variable name over a register when the two collides

coby via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 24 13:58:09 PDT 2017


coby added a comment.

In https://reviews.llvm.org/D34739#819343, @rnk wrote:

> I reverted this in https://reviews.llvm.org/rL308926 because it broke sanitizer-windows: http://lab.llvm.org:8011/builders/sanitizer-windows/builds/14535
>
> With this patch, clang would reject this code:
>
>   namespace cl { }
>   void f() {
>     __asm {
>       mov al, cl
>     }
>   }
>


blimey
my bad
will refine it and re-upload
thanks


Repository:
  rL LLVM

https://reviews.llvm.org/D34739





More information about the llvm-commits mailing list