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

hfinkel at anl.gov hfinkel at anl.gov
Mon Mar 9 13:45:21 PDT 2015


LGTM. Make sure you mention in the commit message that no reduced test case is available.

> As for using std:equal, I do not think it works because it will compare *I1 != *I2 instead of I1 != I2 which we need to compare MemoryOperand references.


Ah, yes, you're right (you could write a custom predicate, but that does not seem worthwhile).


http://reviews.llvm.org/D7769

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






More information about the llvm-commits mailing list