[PATCH] D81515: [llvm] Release-mode ML InlineAdvisor

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 29 09:10:16 PDT 2020


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


================
Comment at: llvm/test/lit.site.cfg.py.in:51
 config.linked_bye_extension = @LLVM_BYE_LINK_INTO_TOOLS@
+config.have_tf_aot = ("@LLVM_HAVE_TF_AOT@" == "ON")
 
----------------
mtrofin wrote:
> thakis wrote:
> > Please use llvm_canonicalize_cmake_booleans for this.
> could you elaborate how? I see it used in CMakeLists files - not super sure how I'd use it here.
> 
> Thanks!
Being addressed in D82776.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81515





More information about the llvm-commits mailing list