[all-commits] [llvm/llvm-project] efd44f: [clang-tidy] Fix crash when diagnostic is emit wit...

Piotr Zegar via All-commits all-commits at lists.llvm.org
Fri Aug 11 09:05:14 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: efd44f80a5a8194b9fe26ff3244ce702cd8dab73
      https://github.com/llvm/llvm-project/commit/efd44f80a5a8194b9fe26ff3244ce702cd8dab73
  Author: Piotr Zegar <me at piotrzegar.pl>
  Date:   2023-08-11 (Fri, 11 Aug 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp
    M clang-tools-extra/test/clang-tidy/infrastructure/diagnostic.cpp

  Log Message:
  -----------
  [clang-tidy] Fix crash when diagnostic is emit with invalid location

Fix crash when diagnostic is emit with invalid location,
but with attached valid ranges. Diagnostic can contain
invalid location, but SourceManager attached to it still
can be valid, use it in such case or fallback to known
SourceManager.

Fixes: #64602

Reviewed By: aaron.ballman

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


  Commit: a3a66de37fc038d46d7146c9dccccfe55f26fd3a
      https://github.com/llvm/llvm-project/commit/a3a66de37fc038d46d7146c9dccccfe55f26fd3a
  Author: Piotr Zegar <me at piotrzegar.pl>
  Date:   2023-08-11 (Fri, 11 Aug 2023)

  Changed paths:
    M clang/docs/analyzer/checkers.rst

  Log Message:
  -----------
  [clang][analyzer][NFC] Change PlacementNewChecker into PlacementNew in documentation

Check name according to Checkers.td is actually a PlacementNew.

Reviewed By: steakhal

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


Compare: https://github.com/llvm/llvm-project/compare/f76ffc1f406e...a3a66de37fc0


More information about the All-commits mailing list