[PATCH] Remove all MMOs from memory operations when tail merging.
hfinkel at anl.gov
hfinkel at anl.gov
Sat Mar 7 20:11:08 PST 2015
================
Comment at: include/llvm/CodeGen/MachineMemOperand.h:203
@@ +202,3 @@
+
+ friend bool operator== (const MachineMemOperand &LHS, const MachineMemOperand &RHS) {
+ return LHS.getValue() == RHS.getValue() &&
----------------
Line too long?
================
Comment at: include/llvm/CodeGen/MachineMemOperand.h:215
@@ +214,3 @@
+
+ friend bool operator!= (const MachineMemOperand &LHS, const MachineMemOperand &RHS) {
+ return !(LHS == RHS);
----------------
Line too long?
================
Comment at: lib/CodeGen/BranchFolding.cpp:737
@@ +736,3 @@
+ return false;
+ for (; I1 != E1; ++I1, ++I2) {
+ if (**I1 != **I2)
----------------
I think this loop is just:
return std::equal(I1, E1, I2);
http://reviews.llvm.org/D7769
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list