[PATCH] D103972: Remove redundant environment variable XLA_FLAGS.

Jacob Hegna via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 9 09:40:41 PDT 2021


jacobhegna created this revision.
jacobhegna added a reviewer: mtrofin.
Herald added a subscriber: mgorny.
jacobhegna requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

If the flag is not set, the script saved_model_aot_compile.py in tensorflow will
default it to the correct value. However, in TF 2.5, the way the value is set in
TensorFlowCompile.cmake file triggers a build error.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D103972

Files:
  llvm/cmake/modules/TensorFlowCompile.cmake


Index: llvm/cmake/modules/TensorFlowCompile.cmake
===================================================================
--- llvm/cmake/modules/TensorFlowCompile.cmake
+++ llvm/cmake/modules/TensorFlowCompile.cmake
@@ -32,7 +32,7 @@
     tfgetmodel(${model} LLVM_ML_MODELS_ABSOLUTE)
     message("Using model at " ${LLVM_ML_MODELS_ABSOLUTE})
     add_custom_command(OUTPUT ${obj_file} ${hdr_file}
-      COMMAND "XLA_FLAGS=\"--xla_cpu_multi_thread_eigen=false\"" ${TENSORFLOW_AOT_COMPILER} aot_compile_cpu
+      COMMAND ${TENSORFLOW_AOT_COMPILER} aot_compile_cpu
             --dir ${LLVM_ML_MODELS_ABSOLUTE}
             --tag_set ${tag_set}
             --signature_def_key ${signature_def_key}


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D103972.350926.patch
Type: text/x-patch
Size: 695 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210609/2386c4d1/attachment.bin>


More information about the llvm-commits mailing list