craig.topper added a comment.

x86 is aggressive about optimizing shuffles no matter where they came from. 
FWIW, InstCombine has a general rule that its not supposed to create a shuffle 
mask that didn't already exist in the IR except for special things like 
identity masks that would allow the shuffle to be removed.  DAG combine is 
supposed to check with TargetLowering::isShuffleMaskLegal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66983



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to