[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