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

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 3 10:14:34 PDT 2017


arsenm accepted this revision.
arsenm added a comment.
This revision is now accepted and ready to land.

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


https://reviews.llvm.org/D32596





More information about the llvm-commits mailing list