[all-commits] [llvm/llvm-project] b7c988: [analyzer][NFC] Introduce the checker package sepa...

[Balazs Benics via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: b7c988811d50f0f068a375292fb5f2b6df384400
      https://github.com/llvm/llvm-project/commit/b7c988811d50f0f068a375292fb5f2b6df384400
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-04-19 (Tue, 19 Apr 2022)

  Changed paths:
    M clang/utils/TableGen/ClangSACheckersEmitter.cpp

  Log Message:
  -----------
  [analyzer][NFC] Introduce the checker package separator character

Reviewed By: martong, ASDenysPetrov

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


  Commit: 63c4ca9d14bad909f92ef019ec85f7a1c42311f2
      https://github.com/llvm/llvm-project/commit/63c4ca9d14bad909f92ef019ec85f7a1c42311f2
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-04-19 (Tue, 19 Apr 2022)

  Changed paths:
    M clang/utils/TableGen/ClangSACheckersEmitter.cpp

  Log Message:
  -----------
  [analyzer] Turn missing tablegen doc entry of a checker into fatal error

It turns out all checkers explicitly mention the `Documentation<>`.
It makes sense to demand this, so emit a fatal tablegen error if such
happens.

Reviewed By: martong, Szelethus

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


  Commit: 744e2a3e2232db87cb68b201739956ca205dc946
      https://github.com/llvm/llvm-project/commit/744e2a3e2232db87cb68b201739956ca205dc946
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-04-19 (Tue, 19 Apr 2022)

  Changed paths:
    M clang/test/Analysis/diagnostics/Inputs/expected-sarif/sarif-multi-diagnostic-test.c.sarif
    M clang/utils/TableGen/ClangSACheckersEmitter.cpp

  Log Message:
  -----------
  [analyzer] ClangSA should tablegen doc urls refering to the main doc page

AFAIK we should prefer
https://clang.llvm.org/docs/analyzer/checkers.html to
https://clang-analyzer.llvm.org/{available_checks,alpha_checks}.html

This patch will ensure that the doc urls produced by tablegen for the
ClangSA, will use the new url. Nothing else will be changed.

Reviewed By: martong, Szelethus, ASDenysPetrov

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


  Commit: 798418982630b5e3acdfc7ba398993aef2071f32
      https://github.com/llvm/llvm-project/commit/798418982630b5e3acdfc7ba398993aef2071f32
  Author: Balazs Benics <balazs.benics at sigmatechnology.se>
  Date:   2022-04-19 (Tue, 19 Apr 2022)

  Changed paths:
    M clang/include/clang/StaticAnalyzer/Checkers/CheckerBase.td
    M clang/include/clang/StaticAnalyzer/Checkers/Checkers.td

  Log Message:
  -----------
  [analyzer] Remove HasAlphaDocumentation tablegen enum value

D121387 simplified the doc url generation process, so we no longer need
the HasAlphaDocumentation enum entry. This patch removes that.

Reviewed By: martong

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


Compare: https://github.com/llvm/llvm-project/compare/68e73eaee632...798418982630