[PATCH] D88371: Introduce a LLVM_ENABLE_TENSORFLOW CMake option for users to opt-in instead of silently trying to enable it

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 28 08:26:48 PDT 2020


mtrofin added a comment.

There are 2 types of TF dependencies, orthogonal from each other, controlled by TENSORFLOW_AOT_PATH and TENSORFLOW_C_LIB_PATH, respectively.

The former should already provide the desired behavior.

The latter is the problem, due to `find_library` behavior (line 840 old code): I think it's missing a NO_DEFAULT_PATH, which should then result in the desired behavior.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88371



More information about the llvm-commits mailing list