[llvm-branch-commits] [llvm] [TRI] Remove reserved registers in getRegPressureSetLimit (PR #118787)
Matt Arsenault via llvm-branch-commits
llvm-branch-commits at lists.llvm.org
Thu Jan 9 08:07:29 PST 2025
================
@@ -925,9 +925,16 @@ class TargetRegisterInfo : public MCRegisterInfo {
virtual const char *getRegPressureSetName(unsigned Idx) const = 0;
/// Get the register unit pressure limit for this dimension.
- /// This limit must be adjusted dynamically for reserved registers.
+ /// TargetRegisterInfo adjusts this limit for reserved registers.
+ /// Avoid using this method directly as it is costly to compute. Use the
+ /// cached version `RegisterClassInfo::getRegPressureSetLimit` instead.
virtual unsigned getRegPressureSetLimit(const MachineFunction &MF,
----------------
arsenm wrote:
If this version depends on dynamic function state, it would be more consistent to move it to MachineRegisterInfo
https://github.com/llvm/llvm-project/pull/118787
More information about the llvm-branch-commits
mailing list