[PATCH] D132658: [InstCombine] Distributive or+mul with const operand

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 18:57:12 PDT 2022


Allen marked an inline comment as done.
Allen added a comment.

In D132658#3751724 <https://reviews.llvm.org/D132658#3751724>, @spatel wrote:

> I think it would be better to make the 'nuw' change separately. That is missing from the current transform, so it could be done before this patch with 'add' alone, or make that a follow-up patch to this one.

Thanks for your suggestion, addressed with D132777 <https://reviews.llvm.org/D132777>


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

https://reviews.llvm.org/D132658



More information about the llvm-commits mailing list