[PATCH] D32596: [DAGCombine] Transform (fadd A, (fmul B, -2.0)) -> (fsub A, (fadd B, B)).

[Chad Rosier via Phabricator via llvm-commits]

mcrosier added a comment.

In https://reviews.llvm.org/D32596#744944, @arsenm wrote:

> In https://reviews.llvm.org/D32596#744882, @mcrosier wrote:
>
> > @arsenm: I don't believe this new version causes the instruction encoding to change in size, but please correct me if I'm wrong.
>
>
> Yes, this is always better if it's really an fsub since the user then doesn't matter


Great.  I'll wait for other's feedback before committing.


https://reviews.llvm.org/D32596