[all-commits] [llvm/llvm-project] 47eec7: [clang][deps] Remove hard-coded resource directory...

Jan Svoboda via All-commits all-commits at lists.llvm.org
Tue Dec 14 02:38:01 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 47eec789ed9cc7d4cb9f1eaa0cafe3e509af015c
      https://github.com/llvm/llvm-project/commit/47eec789ed9cc7d4cb9f1eaa0cafe3e509af015c
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2021-12-14 (Tue, 14 Dec 2021)

  Changed paths:
    R clang/test/ClangScanDeps/Inputs/resource_directory/compiler
    M clang/test/ClangScanDeps/resource_directory.c

  Log Message:
  -----------
  [clang][deps] Remove hard-coded resource directory from tests

Dependency scanner test for resource directory deduction doesn't account for LLVM builds with custom `CLANG_RESOURCE_DIR`.

This patch ensures we don't hardcode the default behavior into the test and take into account the actual value. This is done by running `%clang -print-resource-dir` and using that as the expected value in test assertions.
New comment also clarifies this is different from running that command as part of the dependency scan.

Reviewed By: mgorny

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




More information about the All-commits mailing list