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

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 5 07:47:53 PDT 2017


mcrosier added a comment.

In https://reviews.llvm.org/D32596#745247, @gberry wrote:

> My only comment is that you may be missing additional cases where the fneg could be folded away, but perhaps those can be fixed in a follow up change.


Here's at least one case we're missing: https://bugs.llvm.org/show_bug.cgi?id=32939


Repository:
  rL LLVM

https://reviews.llvm.org/D32596





More information about the llvm-commits mailing list