[PATCH] D100211: [InstCombine] Fold cmpeq/ne(and(~X,Y),0) --> cmpeq/ne(or(freeze(X),Y),X)

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 11 09:08:54 PDT 2021


xbolva00 added a comment.

In D100211#2681688 <https://reviews.llvm.org/D100211#2681688>, @nikic wrote:

> I'm somewhat doubtful that this fold will be beneficial if it requires inserting freeze. Is there some broader context here for which this is intended?

+1. So for now, use isGuaranteedNotToBeUndefOrPoison?


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