[PATCH] D40287: [X86][AVX2]: Adding full coverage of MC encoding for the AVX2 isa set.<NFC>

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 26 01:25:43 PST 2017


RKSimon added a comment.

In https://reviews.llvm.org/D40287#935127, @craig.topper wrote:

> I think he wants you to make sure you test both values of the VEX.R, VEX.X and VEX.B bits.


Yes, that will require testing on the AVX1/AVX2/FMA tests at least, maybe others as well.


Repository:
  rL LLVM

https://reviews.llvm.org/D40287





More information about the llvm-commits mailing list