[PATCH] D115847: [mlgo][inline] Improve global state tracking

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin marked 2 inline comments as done.
mtrofin added inline comments.


================
Comment at: llvm/test/Transforms/Inline/ML/state-tracking-scc-splits.ll:4
+; RUN:   -S -enable-ml-inliner=development -keep-inline-advisor-for-printing < %s 2>&1 | FileCheck %s
+; REQUIRES: have_tf_api
+;
----------------
aeubanks wrote:
> not related to this patch but you could add this as a `lit.local.cfg` in the `ML` directory so you don't have to put it on every test
some tests have `REQUIRES: have_tf_api`, some `have_tf_aot`, and some both. ('api' is for training, 'aot' is for deployment compiler, i.e. where we embed a pre-trained model as a .o)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115847