fix: review

Author: woruyu

llvm/include/llvm/CodeGen/TargetLowering.h

@@ -5681,13 +5681,6 @@ class LLVM_ABI TargetLowering : public TargetLoweringBase {
   /// only the first Count elements of the vector are used.
   SDValue expandVecReduce(SDNode *Node, SelectionDAG &DAG) const;
 
-  /// This implements llvm.canonicalize.f* by multiplication with 1.0, as
-  /// suggested in
-  /// https://llvm.org/docs/LangRef.html#llvm-canonicalize-intrinsic.
-  /// It uses strict_fp operations even outside a strict_fp context in order
-  /// to guarantee that the canonicalization is not optimized away by later
-  /// passes. The result chain introduced by that is intentionally ignored
-  /// since no ordering requirement is intended here.
   SDValue expandFCanonicalizeWithStrictFmul(SDNode *Node, SDLoc DL,
                                             SelectionDAG &DAG) const;
 

llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp

@@ -1310,7 +1310,6 @@ void VectorLegalizer::Expand(SDNode *Node, SmallVectorImpl<SDValue> &Results) {
     }
     break;
   case ISD::FCANONICALIZE: {
-    const TargetLowering &TLI = DAG.getTargetLoweringInfo();
     SDLoc dl(Node);
     SDValue Result = TLI.expandFCanonicalizeWithStrictFmul(Node, dl, DAG);
     Results.push_back(Result);

llvm/lib/CodeGen/SelectionDAG/TargetLowering.cpp

@@ -11584,6 +11584,14 @@ SDValue TargetLowering::expandVecReduce(SDNode *Node, SelectionDAG &DAG) const {
 SDValue
 TargetLowering::expandFCanonicalizeWithStrictFmul(SDNode *Node, SDLoc DL,
                                                   SelectionDAG &DAG) const {
+  // This implements llvm.canonicalize.f* by multiplication with 1.0, as
+  // suggested in
+  // https://llvm.org/docs/LangRef.html#llvm-canonicalize-intrinsic.
+  // It uses strict_fp operations even outside a strict_fp context in order
+  // to guarantee that the canonicalization is not optimized away by later
+  // passes. The result chain introduced by that is intentionally ignored
+  // since no ordering requirement is intended here.
+
   // Create strict multiplication by 1.0.
   SDValue Operand = Node->getOperand(0);
   EVT VT = Operand.getValueType();