[PATCH] Remove all MMOs from memory operations when tail merging.

Ana Pazos apazos at codeaurora.org
Thu Feb 19 20:16:38 PST 2015


================
Comment at: lib/CodeGen/BranchFolding.cpp:735
@@ +734,3 @@
+    if (MII.mayLoad() || MII.mayStore())
+      MII.clearMemRefs();
+}
----------------
hfinkel wrote:
> We can do better than this. Don't remove the MMOs if we're tail merging two instructions that have identical MMOs.
> 
Thanks for the suggestion Hal. Chad is OOO, I will uplload a new patch so we can continue the discussion.

http://reviews.llvm.org/D7769

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






More information about the llvm-commits mailing list