[llvm] [mlir] [MLGO] Do not hardcode 33 eviction feature columns (PR #224598)
Arseniy Obolenskiy via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 1 09:01:57 PDT 2026
================
@@ -24,7 +24,8 @@
]
"""
PER_REGISTER_FEATURE_LIST = ["mask"]
-NUM_REGISTERS = 33
+# Must match getRequiredNumColumns() for the target the tests run on (X86).
+NUM_REGISTERS = 37
----------------
aobolensk wrote:
Reverted back to 33. I don't really understand the 33 number, getNumAllocatableRegs returns 16 and I see even now the most of the numbers are 0s. For now I left it max(32, actual_reg_count) which seem not to regress the x86 case here. But 33 is odd
https://github.com/llvm/llvm-project/pull/224598
More information about the llvm-commits
mailing list