[all-commits] [llvm/llvm-project] 7a5184: [scan-build] Fix clang++ pathname again

[stbergmann via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 7a5184ed951ad38d652d7b3c339f4c6ae7842eeb
      https://github.com/llvm/llvm-project/commit/7a5184ed951ad38d652d7b3c339f4c6ae7842eeb
  Author: Stephan Bergmann <sbergman at redhat.com>
  Date:   2020-11-03 (Tue, 03 Nov 2020)

  Changed paths:
    A clang/test/Analysis/scan-build/cxx-name.test
    M clang/test/Analysis/scan-build/lit.local.cfg
    M clang/tools/scan-build/bin/scan-build

  Log Message:
  -----------
  [scan-build] Fix clang++ pathname again

e00629f777d9d62875730f40d266727df300dbb2 "[scan-build] Fix clang++ pathname" had
removed the -MAJOR.MINOR suffix, but since presumably LLVM 7 the suffix is only
-MAJOR, so ClangCXX (i.e., the CLANG_CXX environment variable passed to
clang/tools/scan-build/libexec/ccc-analyzer) now contained a non-existing
/path/to/clang-12++ (which apparently went largely unnoticed as
clang/tools/scan-build/libexec/ccc-analyzer falls back to just 'clang++' if the
executable denoted by CLANG_CXX does not exist).

For the new clang/test/Analysis/scan-build/cxx-name.test to be effective,
%scan-build must now take care to pass the clang executable's resolved pathname
(i.e., ending in .../clang-MAJOR rather than just .../clang) to --use-analyzer.

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