[PATCH] D105098: [DSE][NFC] Introduce "doesn't overwrite" return code for isOverwrite
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 17 01:03:30 PDT 2021
fhahn requested changes to this revision.
fhahn added a comment.
This revision now requires changes to proceed.
In D105098#2986088 <https://reviews.llvm.org/D105098#2986088>, @ebrevnov wrote:
> In D105098#2984988 <https://reviews.llvm.org/D105098#2984988>, @fhahn wrote:
>
>> Looks good in general, but I couldn't find a use in the linked patches. Could you update the description with more details about the motivation, when/how this will be used.
>
> The thing is that I had to remove depending changes from the list since I didn't manage to come up with a regression test. Looks like it is not that critical and we can return to it once main functionality settles down. So let's skip this one for now as well if you don't mind :-)
Sounds good to me. Marking as changes requested so it is gone from the review queue for now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105098/new/
https://reviews.llvm.org/D105098
More information about the llvm-commits
mailing list