[PATCH] D17668: [BranchFolding] Change function name related with merging MMOs. NFC

Junmo Park via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 2 20:02:15 PST 2016


This revision was automatically updated to reflect the committed changes.
Closed by commit rL262580: [BranchFolding] Change function name related with merging MMOs. NFC (authored by flyingforyou).

Changed prior to commit:
  http://reviews.llvm.org/D17668?vs=49269&id=49701#toc

Repository:
  rL LLVM

http://reviews.llvm.org/D17668

Files:
  llvm/trunk/lib/CodeGen/BranchFolding.cpp

Index: llvm/trunk/lib/CodeGen/BranchFolding.cpp
===================================================================
--- llvm/trunk/lib/CodeGen/BranchFolding.cpp
+++ llvm/trunk/lib/CodeGen/BranchFolding.cpp
@@ -745,11 +745,9 @@
 }
 
 static void
-removeMMOsFromMemoryOperations(MachineBasicBlock::iterator MBBIStartPos,
-                               MachineBasicBlock &MBBCommon) {
-  // Remove MMOs from memory operations in the common block
-  // when they do not match the ones from the block being tail-merged.
-  // This ensures later passes conservatively compute dependencies.
+mergeMMOsFromMemoryOperations(MachineBasicBlock::iterator MBBIStartPos,
+                              MachineBasicBlock &MBBCommon) {
+  // Merge MMOs from memory operations in the common block.
   MachineBasicBlock *MBB = MBBIStartPos->getParent();
   // Note CommonTailLen does not necessarily matches the size of
   // the common BB nor all its instructions because of debug
@@ -900,8 +898,8 @@
         continue;
       DEBUG(dbgs() << "BB#" << SameTails[i].getBlock()->getNumber()
                    << (i == e-1 ? "" : ", "));
-      // Remove MMOs from memory operations as needed.
-      removeMMOsFromMemoryOperations(SameTails[i].getTailStartPos(), *MBB);
+      // Merge MMOs from memory operations as needed.
+      mergeMMOsFromMemoryOperations(SameTails[i].getTailStartPos(), *MBB);
       // Hack the end off BB i, making it jump to BB commonTailIndex instead.
       ReplaceTailWithBranchTo(SameTails[i].getTailStartPos(), MBB);
       // BB i is no longer a predecessor of SuccBB; remove it from the worklist.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17668.49701.patch
Type: text/x-patch
Size: 1615 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160303/ad32f09e/attachment.bin>


More information about the llvm-commits mailing list