[llvm] bb6ab8b - [TargetLowering] Update the description of getShiftAmountTy. NFC

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 4 09:28:06 PDT 2024


Author: Craig Topper
Date: 2024-07-04T09:27:59-07:00
New Revision: bb6ab8b4f491cb1eaf6a88ca0b0bb283e3bd8bba

URL: https://github.com/llvm/llvm-project/commit/bb6ab8b4f491cb1eaf6a88ca0b0bb283e3bd8bba
DIFF: https://github.com/llvm/llvm-project/commit/bb6ab8b4f491cb1eaf6a88ca0b0bb283e3bd8bba.diff

LOG: [TargetLowering] Update the description of getShiftAmountTy. NFC

After f4d058fdb1f456fdab9593eab4955a62b6bb8e70, the LegalTypes
argument is no longer used.

Added: 
    

Modified: 
    llvm/include/llvm/CodeGen/TargetLowering.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/CodeGen/TargetLowering.h b/llvm/include/llvm/CodeGen/TargetLowering.h
index 9a0df8b29d752..51ff7855ec046 100644
--- a/llvm/include/llvm/CodeGen/TargetLowering.h
+++ b/llvm/include/llvm/CodeGen/TargetLowering.h
@@ -400,12 +400,10 @@ class TargetLoweringBase {
   virtual MVT getScalarShiftAmountTy(const DataLayout &, EVT) const;
 
   /// Returns the type for the shift amount of a shift opcode. For vectors,
-  /// returns the input type. For scalars, behavior depends on \p LegalTypes. If
-  /// \p LegalTypes is true, calls getScalarShiftAmountTy, otherwise uses
-  /// pointer type. If getScalarShiftAmountTy or pointer type cannot represent
-  /// all possible shift amounts, returns MVT::i32. In general, \p LegalTypes
-  /// should be set to true for calls during type legalization and after type
-  /// legalization has been completed.
+  /// returns the input type. For scalars, calls getScalarShiftAmountTy.
+  /// If getScalarShiftAmountTy type cannot represent all possible shift
+  /// amounts, returns MVT::i32.
+  /// \p LegalTypes is no longer used and will be removed from the interface.
   EVT getShiftAmountTy(EVT LHSTy, const DataLayout &DL,
                        bool LegalTypes = true) const;
 


        


More information about the llvm-commits mailing list