[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