Simplify comments

Author: adelejjeh

llvm/lib/Target/AMDGPU/AMDGPUISelLowering.cpp

@@ -2806,9 +2806,8 @@ SDValue AMDGPUTargetLowering::LowerFLOGCommon(SDValue Op,
     SDValue C = DAG.getConstantFP(IsLog10 ? c_log10 : c_log, DL, VT);
     SDValue CC = DAG.getConstantFP(IsLog10 ? cc_log10 : cc_log, DL, VT);
     // Our implementation of LOG is not contract safe because we generate
-    // error-correcting summations based on the rounding error of the first
-    // multiplication below, so contracting the multiply with the final add will
-    // lead to inaccurate final results. Disable contraction for the expanded
+    // error-correcting summations for which contraction may lead to an increase
+    // in the error of the approximation. Disable contraction for the expanded
     // instructions.
     Flags.setAllowContract(false);
     R = DAG.getNode(ISD::FMUL, DL, VT, Y, C, Flags);
@@ -2834,9 +2833,8 @@ SDValue AMDGPUTargetLowering::LowerFLOGCommon(SDValue Op,
     SDValue YH = DAG.getNode(ISD::BITCAST, DL, MVT::f32, YHInt);
     SDValue YT = DAG.getNode(ISD::FSUB, DL, VT, Y, YH, Flags);
     // Our implementation of LOG is not contract safe because we generate
-    // error-correcting summations based on the rounding error of the first
-    // multiplication below, so contracting the multiply with the final add will
-    // lead to inaccurate final results. Disable contraction for the expanded
+    // error-correcting summations for which contraction may lead to an increase
+    // in the error of the approximation. Disable contraction for the expanded
     // instructions.
     Flags.setAllowContract(false);
     SDValue YTCT = DAG.getNode(ISD::FMUL, DL, VT, YT, CT, Flags);

llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.cpp

@@ -3552,10 +3552,9 @@ bool AMDGPULegalizerInfo::legalizeFlogCommon(MachineInstr &MI,
     auto C = B.buildFConstant(Ty, IsLog10 ? c_log10 : c_log);
     auto CC = B.buildFConstant(Ty, IsLog10 ? cc_log10 : cc_log);
     // Our implementation of LOG is not contract safe because we generate
-    // error-correcting summations based on the rounding error of the first
-    // multiplication below, so contracting the multiply with the final add will
-    // lead to inaccurate final results. Disable contraction for the expanded
-    // instructions.
+    // error-correcting summations  for which contraction may lead to an
+    // increase in the error of the approximation. Disable contraction for the
+    // expanded instructions.
     auto NewFlags = Flags & ~(MachineInstr::FmContract);
     R = B.buildFMul(Ty, Y, C, NewFlags).getReg(0);
     auto NegR = B.buildFNeg(Ty, R, NewFlags);
@@ -3578,10 +3577,9 @@ bool AMDGPULegalizerInfo::legalizeFlogCommon(MachineInstr &MI,
     auto YH = B.buildAnd(Ty, Y, MaskConst);
     auto YT = B.buildFSub(Ty, Y, YH, Flags);
     // Our implementation of LOG is not contract safe because we generate
-    // error-correcting summations based on the rounding error of the first
-    // multiplication below, so contracting the multiply with the final add will
-    // lead to inaccurate final results. Disable contraction for the expanded
-    // instructions.
+    // error-correcting summations  for which contraction may lead to an
+    // increase in the error of the approximation. Disable contraction for the
+    // expanded instructions.
     auto NewFlags = Flags & ~(MachineInstr::FmContract);
     auto YTCT = B.buildFMul(Ty, YT, CT, NewFlags);