[PATCH] D109194: [InstCombine] Optimize (a & b) | (~a & c)

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 2 18:32:49 PDT 2021


MatzeB added a subscriber: echristo.
MatzeB added a comment.

So @lebedev.ri if I did my history research correctly, then the pattern in my patch here has also been part of https://reviews.llvm.org/D46814. That diff however was reverted in the meantime (see https://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20180528/556578.html @echristo) because of undef behavior? My version here adds a `freeze` instruction (thanks to alive2.llvm.org for catching this or I would have missed that subtlety too!).

So maybe it this is ok now?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109194/new/

https://reviews.llvm.org/D109194



More information about the llvm-commits mailing list