[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