[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