[PATCH] [X86][AVX] Fix wrong lowering of VPERM2X128 nodes.

Michael Kuperstein michael.m.kuperstein at intel.com
Sun Mar 8 08:52:18 PDT 2015


I may be missing something, but I don't see how this helps.
We'll still get a nonsense MaskLO if both Mask[0] and Mask[1] are SM_SentinelUndef, and SM_SentinelUndef happens to be -INT_MAX.
Why not set MaskLO/MaskHI to 0 explicitly if both relevant Mask elements are undef?

(I know this is purely theoretical, if you feel it unnecessarily complicates the code, feel free to ignore.)


http://reviews.llvm.org/D8119

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list