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

Alexander Kornienko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 10:23:29 PDT 2022


alexfh added a comment.

In D124119#3556322 <https://reviews.llvm.org/D124119#3556322>, @alexfh wrote:

> In D124119#3550149 <https://reviews.llvm.org/D124119#3550149>, @bipmis wrote:
>
>> @alexfh I think it would be good if you can try the patch and approve for commit if it looks fine. Thanks.
>
> I have verified clang with your latest patch on the original file, and it doesn't hang now. However, I'm not an expert in InstCombine and can't properly review the change.

And yes, please add a regression test.


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

https://reviews.llvm.org/D124119



More information about the llvm-commits mailing list