[all-commits] [llvm/llvm-project] c29513: [clangd] Fix check-clangd with no clang built

Sam McCall via All-commits all-commits at lists.llvm.org
Mon Nov 2 12:11:04 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: c29513f7e023f125c6d221db179dc40b79e5c074
      https://github.com/llvm/llvm-project/commit/c29513f7e023f125c6d221db179dc40b79e5c074
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2020-11-02 (Mon, 02 Nov 2020)

  Changed paths:
    M clang-tools-extra/clangd/test/document-link.test
    M clang-tools-extra/clangd/test/lit.cfg.py
    M llvm/utils/lit/lit/llvm/config.py

  Log Message:
  -----------
  [clangd] Fix check-clangd with no clang built

- pass required=False to use_clang(), as we don't need it
- fix required=False (which was unused and rotted):
  - make derived substitutions conditional on it
  - add a feature so we can disable tests that need it
- conditionally disable our one test that depends on %resource_dir.
  This doesn't seem right from first principles, but isn't a big deal.

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




More information about the All-commits mailing list