[PATCH] D135780: [IR] Switch everything to use memory attribute
Johannes Doerfert via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 21 10:32:45 PDT 2022
jdoerfert added a comment.
In D135780#3874753 <https://reviews.llvm.org/D135780#3874753>, @nikic wrote:
> In D135780#3874732 <https://reviews.llvm.org/D135780#3874732>, @jdoerfert wrote:
>
>> In D135780#3874715 <https://reviews.llvm.org/D135780#3874715>, @nikic wrote:
>>
>>> The only missing part now is Attributor support.
>>
>> What is missing there?
>
> The interaction of AAMemoryBehavior and AAMemoryLocation with the IR attributes. They need to initialized based on the memory attribute now and also manifest their results that way.
You want me to look into it? I didn't expect this to be much more complicated than the FunctionAttr pass. Maybe I'm wrong.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135780/new/
https://reviews.llvm.org/D135780
More information about the llvm-commits
mailing list