[PATCH] D131930: [mlgo] Add in-development instruction based features for regalloc advisor

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 16:24:31 PDT 2022


mtrofin added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/SlotIndexes.h:148
 
+    void setEntry(IndexListEntry *Entry) {
+      lie.setPointer(Entry);
----------------
(just noticed this) can you avoid this, because it's just for test, and instead just replace the SlotIndex in your collection?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D131930/new/

https://reviews.llvm.org/D131930



More information about the llvm-commits mailing list