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

hfinkel at anl.gov hfinkel at anl.gov
Wed Mar 4 11:34:46 PST 2015


================
Comment at: lib/CodeGen/BranchFolding.cpp:740
@@ +739,3 @@
+  for (; MMOI1 != MMOE1; ++MMOI1, ++MMOI2) {
+    const Value *V1 = (*MMOI1)->getValue();
+    const Value *V2 = (*MMOI2)->getValue();
----------------
Can you please add an == operator to the class; this logic needs to be maintained with the MMO class, and should not live here.

http://reviews.llvm.org/D7769

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






More information about the llvm-commits mailing list