[PATCH] D119049: [LLD] Allow usage of LLD as a library

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 16 15:31:42 PDT 2023


MaskRay added a comment.

In D119049#4429521 <https://reviews.llvm.org/D119049#4429521>, @aganea wrote:

> With binary files.

I see that

  config.environment["LLD_SRC_DIR"] = config.lld_src_dir

sets the correct path, but `ninja check-lld-unit` still fails due to `thisPath.append(getenv("LLD_SRC_DIR"));` getting an empty string...



================
Comment at: lld/test/Unit/lit.cfg.py:26
+# if none of these are present.
+if 'TMP' in os.environ:
+    config.environment['TMP'] = os.environ['TMP']
----------------
Double quotes to be consistent with https://reviews.llvm.org/D150545


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119049



More information about the llvm-commits mailing list