[PATCH] D26586: [AVX512][inline-asm] Fix AVX512 inline assembly instruction resolution when the size qualifier of a memory operand is not specified explicitly.

Reid Kleckner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 4 09:41:48 PDT 2017


rnk added a comment.

In https://reviews.llvm.org/D26586#746128, @rnk wrote:

> I feel like we need to step back and ask why we are using the variable size information provided by the frontend at all. I don't think MSVC uses it. We probably only needed it because our intel asm matcher couldn't cope with ambiguity before and it was getting the wrong sizes.
>
> I'm going to look into removing the frontend-directed AOK_SizeDirective rewrite altogether.


I looked into this more and commented on the new patch https://reviews.llvm.org/D32636


Repository:
  rL LLVM

https://reviews.llvm.org/D26586





More information about the llvm-commits mailing list