[all-commits] [llvm/llvm-project] f4b5d5: Add use_default_shell_env = True to ctx.actions.run

Alexander Grund via All-commits all-commits at lists.llvm.org
Mon Sep 20 10:09:58 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f4b5d597d86abafd61d4de6235f724d4f7b046e6
      https://github.com/llvm/llvm-project/commit/f4b5d597d86abafd61d4de6235f724d4f7b046e6
  Author: Alexander Grund <alexander.grund at tu-dresden.de>
  Date:   2021-09-20 (Mon, 20 Sep 2021)

  Changed paths:
    M utils/bazel/llvm-project-overlay/mlir/tblgen.bzl

  Log Message:
  -----------
  Add use_default_shell_env = True to ctx.actions.run

When building a tool in a non-standard environment (e.g. custom
compiler path -> LD_LIBRARY_PATH set) then
`use_default_shell_env = True` is required to run that tool in the same
environment or otherwise the build will fail due to missing symbols.
See https://github.com/google/jax/issues/7842 for this issue and
https://github.com/tensorflow/tensorflow/pull/44549 for related fix in
TF.

Reviewed By: GMNGeoffrey

Differential Revision: https://reviews.llvm.org/D109873




More information about the All-commits mailing list