[PATCH] D32596: [InstCombine] Transform fmul X, +2.0 --> fadd X, X.

[Chad Rosier via Phabricator via llvm-commits]

mcrosier added a comment.

In https://reviews.llvm.org/D32596#740344, @efriedma wrote:

> Another data-point: reassociate currently prefers to canonicalize "fadd %x, %x", to "fmul %x, 2".  We don't want to fight back and forth in IR.


Thanks, Eli.  I'll work on extending the DAG combine in that case.  Does that sound reasonable?


https://reviews.llvm.org/D32596