[PATCH] D16875: MemorySSA Optimizations: Patch 1 of N

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 3 21:38:57 PST 2016


majnemer added a subscriber: majnemer.

================
Comment at: lib/Transforms/Utils/MemorySSA.cpp:940
@@ +939,3 @@
+  //
+  // TODO(gbiv): We should handle invariant groups, as well. It's a bit harder,
+  // because we need to pay close attention to invariant group barriers.
----------------
TODO(username) is more of a Google-ism, I don't think LLVM does this very often.

================
Comment at: lib/Transforms/Utils/MemorySSA.cpp:942-943
@@ +941,4 @@
+  // because we need to pay close attention to invariant group barriers.
+  if (isa<LoadInst>(I) && (I->getMetadata(LLVMContext::MD_invariant_load) ||
+       AA->pointsToConstantMemory(I))) {
+    doCacheInsert(StartingAccess, MSSA->getLiveOnEntryDef(), Q, Q.StartingLoc);
----------------
Is this clang-format'd?


http://reviews.llvm.org/D16875





More information about the llvm-commits mailing list