[PATCH] D77752: [llvm] Machine Learned policy for inlining -Oz

[Petr Hosek via Phabricator via llvm-commits]

phosek added inline comments.


================
Comment at: llvm/lib/Analysis/ML/CMakeLists.txt:19
+  target_sources(LLVMMLPolicies PUBLIC $<TARGET_OBJECTS:LLVMMLPoliciesAOT>)
+  target_link_libraries(LLVMMLPolicies  ${tensorflow_c_api})
+endif ()
----------------
I think line 19 and line 25 should be swapped: TensorFlow C API should be used with `LLVM_HAVE_TF_API` and `LLVMMLPoliciesXLA` with `LLVM_HAS_TF_AOT`. I got link errors with the current version, but everything works after swapping those two lines.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77752