[PATCH] Minor tweak to MDA

JF Bastien jfb at chromium.org
Fri Feb 13 14:27:36 PST 2015


lgtm otherwise.


================
Comment at: lib/Analysis/MemoryDependenceAnalysis.cpp:426
@@ -425,2 +425,3 @@
   // Walk backwards through the basic block, looking for dependencies.
-  while (ScanIt != BB->begin()) {
+  // We can stop before processing PHIs or dbg intrinsics
+  const BasicBlock::iterator Begin(BB->getFirstNonPHIOrDbg());
----------------
Missing period.

http://reviews.llvm.org/D7624

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list