[PATCH] D13519: [MemCpyOpt] Fix wrong merging of adjacent nontemporal stores into memset calls.

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 7 12:25:25 PDT 2015


qcolombet added inline comments.

================
Comment at: lib/Transforms/Scalar/MemCpyOptimizer.cpp:495
@@ +494,3 @@
+  if (SI->getMetadata(LLVMContext::MD_nontemporal))
+    return false;
+
----------------
Would it make sense tp extend the metadata to allow this propagation to happen?
I believe the problem here is that we need to differentiate between the load or the store being non-temporal.


http://reviews.llvm.org/D13519





More information about the llvm-commits mailing list