[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