[PATCH] D52140: [X86] Add initial SimplifyDemandedVectorEltsForTargetNode support

Nirav Dave via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 18 10:28:02 PDT 2018


niravd added a comment.

Looks google fine modulo the noted issues with load.



================
Comment at: lib/Target/X86/X86ISelLowering.cpp:31800
+    APInt SrcElts = APInt::getNullValue(NumElts);
+    for (int i = 0; i != NumElts; ++i) {
+      int M = OpMask[i];
----------------
Use OpMask[i]-Lo for M and merge the nested if conditions.


Repository:
  rL LLVM

https://reviews.llvm.org/D52140





More information about the llvm-commits mailing list