[PATCH] D31767: [LLVM][X86][SSE] Update MOVNTDQA non-temporal loads to generic implementation

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: lib/IR/AutoUpgrade.cpp:1878
+      Module *M = F->getParent();
+      SmallVector<Metadata *, 1> Elts;
+      Elts.push_back(
----------------
I know we already do this in several places in this file, but why do we need a SmallVector of 1? Won't a single Metadata * variable auto convert to ArrayRef for the MDNode::get call?


================
Comment at: lib/IR/AutoUpgrade.cpp:1884
+      Value *Ptr = CI->getArgOperand(0);
+      VectorType *VTy = cast<VectorType>(CI->getType());
+
----------------
Do we need a VectorType for the two calls that use this or could it just be generic Type*?


Repository:
  rL LLVM

https://reviews.llvm.org/D31767