[PATCH] Remove all MMOs from memory operations when tail merging.
hfinkel at anl.gov
hfinkel at anl.gov
Mon Feb 23 11:21:22 PST 2015
In http://reviews.llvm.org/D7769#128094, @apazos wrote:
> Hi Hal,
> I added all possible fields so we can agree on what we really need to check.
> I think some of the fields we do need to check, like alignment, address space, flags.
> Others like AAInfo, Ranges, I just checked the first entry, I did not go down the lists to check each metadata node. Probably we need to do that.
> Let me know what you think.
What you're doing with the Metadata right now is correct; you're comparing the pointers to the metadata, and those should be unique by contents (you won't have two different metadata pointers that have the same contents).
http://reviews.llvm.org/D7769
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list