[PATCH] D44548: [DAGCombiner] Expand combining of FP logical operations to sign-setting FP operations

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 31 12:07:52 PDT 2018


efriedma added a comment.

Thinking about it a bit more, we probably want to canonicalize to fabs+fneg anyway on AArch64, rather than the other way, because the fneg could combine with a later operation.  And pattern-matching it is pretty easy, anyway.


Repository:
  rL LLVM

https://reviews.llvm.org/D44548





More information about the llvm-commits mailing list