[PATCH] D118707: [nfc][regalloc] Make the max inference cutoff configurable

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 2 07:34:03 PST 2022


mstorsjo added inline comments.


================
Comment at: llvm/lib/CodeGen/MLRegallocEvictAdvisor.cpp:63
+
 #endif // #ifdef LLVM_HAVE_TF_API
 
----------------
This broke compilation for me:
```
../lib/CodeGen/MLRegallocEvictAdvisor.cpp: In member function ‘bool {anonymous}::MLEvictAdvisor::loadInterferenceFeatures(llvm::LiveInterval&, llvm::MCRegister, bool, const SmallVirtRegSet&, {anonymous}::FeaturesListNormalizer&, size_t) const’:
../lib/CodeGen/MLRegallocEvictAdvisor.cpp:549:50: error: ‘EvictInterferenceCutoff’ was not declared in this scope
  549 |     const auto &IFIntervals = Q.interferingVRegs(EvictInterferenceCutof );
      |                                                  ^~~~~~~~~~~~~~~~~~~~~~
../lib/CodeGen/MLRegallocEvictAdvisor.cpp: At global scope:

```

I guess that's because `EvictInterferenceCutoff` is declared within an ifdef that isn't taken in my build configuration.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118707



More information about the llvm-commits mailing list