[PATCH] D129254: [mlgo] Simplify autogenerated regalloc model
Aiden Grossman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 7 00:20:42 PDT 2022
aidengrossman added reviewers: mtrofin, yundiqian.
aidengrossman added a comment.
This patch removes all unnecessary features from the autogenerated regalloc model (simplifying it greatly) and gets rid of the nondeterminism that I was experiencing that trips the mask==1 assertion in the MLRegallocAdvisor. Not anything super critical, but slightly annoying during development as it has to be commented out and commented back in after feature extraction development is completed and doesn't do what it it is supposed to during development. Tested locally as working and all the tests in `/llvm/test/CodeGen/MLRegalloc` pass for me as well.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D129254/new/
https://reviews.llvm.org/D129254
More information about the llvm-commits
mailing list