[PATCH] D32536: Extend memcpy expansion in Transform/Utils to handle wider operand types.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 26 16:12:49 PDT 2017


arsenm added inline comments.


================
Comment at: lib/Transforms/Utils/LowerMemIntrinsics.cpp:445
+  // Original implementation
+  if (!TTI.useWideIRMemcpyLoopLowering()) {
+    createMemCpyLoop(/* InsertBefore */ Memcpy,
----------------
I'm not sure this is a useful hook. If it's worth keeping the old path that had no target information, making TTI an optional pointer input and using it if available makes more sense


Repository:
  rL LLVM

https://reviews.llvm.org/D32536





More information about the llvm-commits mailing list