[PATCH] D28622: DAGCombiner: Allow negating ConstantFP after legalize

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 24 16:59:34 PST 2017


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

Comment needs to be reworded (I provided a suggestion). Otherwise, LGTM.



================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:614
+
+    // Don't invert constant FP values after legalize, unless the target the
+    // negated constant is necessarily legal.
----------------
 // Don't invert constant FP values after legalization unless the target says the negated constant is legal.


https://reviews.llvm.org/D28622





More information about the llvm-commits mailing list