[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