From 32176133fa29389fdffe37bb26630d883ad1ecdf Mon Sep 17 00:00:00 2001 From: Simon Pilgrim Date: Thu, 13 Feb 2020 13:56:23 +0000 Subject: [PATCH] Move FIXME to start of comment so visual studio actually tags it. NFC. --- llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp index b840fd4..78e0820 100644 --- a/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp +++ b/llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp @@ -13399,9 +13399,10 @@ SDValue DAGCombiner::visitFNEG(SDNode *N) { TargetLowering::NegatibleCost::Expensive) return TLI.getNegatedExpression(N0, DAG, LegalOperations, ForCodeSize); - // -(X-Y) -> (Y-X) is unsafe because when X==Y, -0.0 != +0.0 FIXME: This is - // duplicated in getNegatibleCost, but getNegatibleCost doesn't know it - // was called from a context with a nsz flag if the input fsub does not. + // -(X-Y) -> (Y-X) is unsafe because when X==Y, -0.0 != +0.0 + // FIXME: This is duplicated in getNegatibleCost, but getNegatibleCost doesn't + // know it was called from a context with a nsz flag if the input fsub does + // not. if (N0.getOpcode() == ISD::FSUB && (DAG.getTarget().Options.NoSignedZerosFPMath || N->getFlags().hasNoSignedZeros()) && N0.hasOneUse()) { -- 2.7.4