[PATCH] D100211: [InstCombine] Fold cmpeq/ne(and(~X,Y),0) --> cmpeq/ne(or(freeze(X),Y),freeze(X)) (PR44136)
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 12 05:05:40 PDT 2021
RKSimon added a comment.
In D100211#2682728 <https://reviews.llvm.org/D100211#2682728>, @xbolva00 wrote:
> In D100211#2682640 <https://reviews.llvm.org/D100211#2682640>, @RKSimon wrote:
>
>> Abandoning for now, we can revisit this if/when people get more relaxed about using freeze()
>
> Can you create some umbrella PR for “transformations with freeze” (or something like that) on bugzilla ?
> And add this there.
https://bugs.llvm.org/show_bug.cgi?id=49930
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100211/new/
https://reviews.llvm.org/D100211
More information about the llvm-commits
mailing list