From: Simon Pilgrim Date: Thu, 13 Feb 2020 13:56:23 +0000 (+0000) Subject: Move FIXME to start of comment so visual studio actually tags it. NFC. X-Git-Tag: llvmorg-12-init~14803 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=32176133fa29389fdffe37bb26630d883ad1ecdf;p=platform%2Fupstream%2Fllvm.git Move FIXME to start of comment so visual studio actually tags it. NFC. --- 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()) {