[PATCH] D135592: [IR] Add support for memory attribute

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 17 02:37:40 PDT 2022


nikic added a comment.

In D135592#3861643 <https://reviews.llvm.org/D135592#3861643>, @nhaehnle wrote:

> Am I blind or does this not actually contain the bitcode parts yet? Generally looks fine to me at a glance, I expect you're going to rebase on top of the `MemoryEffects` rename.

Ooops, I uploaded the wrong patch during the last update. Any yes, this will need to be adjusted for the MemoryEffects rename, and also for some of the LangRef changes (`memory()` should be rejected and the `other` location kind dropped).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135592/new/

https://reviews.llvm.org/D135592



More information about the llvm-commits mailing list