[PATCH] D124119: [InstCombine] Combine instructions of type or/and where AND masks can be combined.

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 29 09:03:02 PDT 2022


spatel added a comment.

In D124119#3480733 <https://reviews.llvm.org/D124119#3480733>, @bipmis wrote:

> I have modified and committed the tests with modifications requested. Please have a look if this looks ok. Subsequently will rebase my patch.

The changes to prevent commutes look right. I suspect we'll need even more one-use test variations to be sure we have that right, but that should not hold up rebasing/changing the patch.


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

https://reviews.llvm.org/D124119



More information about the llvm-commits mailing list