[PATCH] D31396: [X86][LLVM][Canonical Compare Intrinsics] Creating a canonical representation for X86 CMP intrinsics

[michael zuckerman via Phabricator via llvm-commits]

m_zuckerman added inline comments.


================
Comment at: lib/Transforms/InstCombine/InstCombineCalls.cpp:1563
+// Example(a == b) -> (b == a)
+// Target must have VEX encoding!!!
+
----------------
craig.topper wrote:
> How are you ensuring the VEX encoding is valid?
I added assert check that intrinsics contains avx. 


https://reviews.llvm.org/D31396