[PATCH] D38736: [X86] test/testn intrinsics lowering to IR. llvm part.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 6 10:45:11 PST 2017


craig.topper accepted this revision.
craig.topper added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/IR/AutoUpgrade.cpp:847
+  return Builder.CreateBitCast(Vec,
+                               IntegerType::get(Context, std::max(NumElts, 8U)));
+}
----------------
You can use Builder.getIntNTy(std::max(NumElts, 8U)) here and you won't need the Context argument. Builder already has a reference to the Context internally and getIntNTy is just a helper to call IntegerType::get using that reference


https://reviews.llvm.org/D38736





More information about the llvm-commits mailing list