[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