[PATCH] D115274: [IR][RFC] Memory region declaration intrinsic

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 23 13:31:50 PDT 2022


lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.

In D115274#3403284 <https://reviews.llvm.org/D115274#3403284>, @jdoerfert wrote:

> In D115274#3361864 <https://reviews.llvm.org/D115274#3361864>, @lebedev.ri wrote:
>
>> Does anybody else have any thoughts/opinions on this?
>>
>> In D115274#3361333 <https://reviews.llvm.org/D115274#3361333>, @jdoerfert wrote:
>>
>>> I still believe this should be a generic assume.passthrough intrinsic with an operand bundle for the specific use case.
>>
>> Is that a blocker?
>
> No. Though, we should put a TODO somewhere if someone comes looking for a nice project to cleanup our messes.

Awesome! There has been a lot of approval of the proposal, but does anyone want to rubber-stamp it before i land this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115274



More information about the llvm-commits mailing list