[PATCH] D133788: [InstCombine] try multi-use demanded bits folds for 'add'

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

In D133788#3787026 <https://reviews.llvm.org/D133788#3787026>, @spatel wrote:

> In D133788#3786981 <https://reviews.llvm.org/D133788#3786981>, @RKSimon wrote:
>
>> Nice - have you looked if DAG SimplifyMultipleUseDemandedBits would benefit?
>
> I saw that we don't have this in codegen either, but I didn't try to adapt it yet. We can do the same thing for Op1 (RHS) of a sub, so that's another potential follow-up.

D133799 <https://reviews.llvm.org/D133799> shows the DAG version...looks like a slog to make that worthwhile.


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

https://reviews.llvm.org/D133788