[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