[PATCH] D59266: [X86] Add support for {vex2}, {vex3}, and {evex} to the assembler to match gas. Use {evex} to improve the one our 32-bit AVX512 tests.

Tue Apr 9 11:40:09 PDT 2019

craig.topper marked an inline comment as done.
craig.topper added inline comments.

================
Comment at: test/MC/X86/AVX-32.s:9
+// CHECK: encoding: [0xc4,0xe1,0x71,0x58,0x8c,0x82,0x10,0xe3,0x0f,0xe3]
+{vex3} vaddpd -485498096(%edx,%eax,4), %xmm1, %xmm1 
+
----------------
RKSimon wrote:
> I take it you will be adding full vex3 testing as a separate (bulky) commit?
Maybe if I can get my hands on the script that generated these tests. Updating the scalar test was painful. Also need to figure out what happened to the effort to get the rest of the reviews committed.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D59266/new/

https://reviews.llvm.org/D59266