[llvm] 767ba58 - [RISCV] Make getMinRVVVectorSizeInBits and getMaxRVVVectorSizeInBits protected [nfc]
Philip Reames via llvm-commits
llvm-commits at lists.llvm.org
Sat Jun 25 11:11:50 PDT 2022
Author: Philip Reames
Date: 2022-06-25T11:11:31-07:00
New Revision: 767ba58f808de8099294bb95929bd8ee55b49da4
URL: https://github.com/llvm/llvm-project/commit/767ba58f808de8099294bb95929bd8ee55b49da4
DIFF: https://github.com/llvm/llvm-project/commit/767ba58f808de8099294bb95929bd8ee55b49da4.diff
LOG: [RISCV] Make getMinRVVVectorSizeInBits and getMaxRVVVectorSizeInBits protected [nfc]
These are now only used in the implementation of getRealMinVLen and getRealMaxVLEn, and useRVVForFixedLengthVectors; make them protected to discourage new users.
Added:
Modified:
llvm/lib/Target/RISCV/RISCVSubtarget.h
Removed:
################################################################################
diff --git a/llvm/lib/Target/RISCV/RISCVSubtarget.h b/llvm/lib/Target/RISCV/RISCVSubtarget.h
index e8dc9b31d91c3..e5c347da3e7cf 100644
--- a/llvm/lib/Target/RISCV/RISCVSubtarget.h
+++ b/llvm/lib/Target/RISCV/RISCVSubtarget.h
@@ -240,6 +240,13 @@ class RISCVSubtarget : public RISCVGenSubtargetInfo {
std::unique_ptr<LegalizerInfo> Legalizer;
std::unique_ptr<RegisterBankInfo> RegBankInfo;
+ // Return the known range for the bit length of RVV data registers as set
+ // at the command line. A value of 0 means nothing is known about that particular
+ // limit beyond what's implied by the architecture.
+ // NOTE: Please use getRealMinVLen and getRealMaxVLen instead!
+ unsigned getMaxRVVVectorSizeInBits() const;
+ unsigned getMinRVVVectorSizeInBits() const;
+
public:
const CallLowering *getCallLowering() const override;
InstructionSelector *getInstructionSelector() const override;
@@ -252,11 +259,6 @@ class RISCVSubtarget : public RISCVGenSubtargetInfo {
// pool if exceeded.
unsigned getMaxBuildIntsCost() const;
- // Return the known range for the bit length of RVV data registers. A value
- // of 0 means nothing is known about that particular limit beyond what's
- // implied by the architecture.
- unsigned getMaxRVVVectorSizeInBits() const;
- unsigned getMinRVVVectorSizeInBits() const;
unsigned getMaxLMULForFixedLengthVectors() const;
bool useRVVForFixedLengthVectors() const;
More information about the llvm-commits
mailing list