[llvm] [mlir] [MLGO] Do not hardcode 33 eviction feature columns (PR #224598)

Mircea Trofin via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 17:06:33 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
----------------
mtrofin wrote:

interesting - so we had a bug here? Our prod policy for x86 has indeed 33 columns. 

https://github.com/llvm/llvm-project/pull/224598


More information about the llvm-commits mailing list