[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