[all-commits] [llvm/llvm-project] 371ecc: [clang-query] Remove support for srcloc output (#9...

Aaron Ballman via All-commits all-commits at lists.llvm.org
Fri May 17 05:07:08 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 371eccd5dfed88c8e76449233d8388c12be3464b
      https://github.com/llvm/llvm-project/commit/371eccd5dfed88c8e76449233d8388c12be3464b
  Author: Aaron Ballman <aaron at aaronballman.com>
  Date:   2024-05-17 (Fri, 17 May 2024)

  Changed paths:
    M clang-tools-extra/clang-query/CMakeLists.txt
    M clang-tools-extra/clang-query/Query.cpp
    M clang-tools-extra/clang-query/Query.h
    M clang-tools-extra/clang-query/QueryParser.cpp
    M clang-tools-extra/clang-query/QuerySession.h
    M clang-tools-extra/docs/ReleaseNotes.rst
    M clang-tools-extra/unittests/clang-query/QueryParserTest.cpp
    M clang/docs/tools/clang-formatted-files.txt
    R clang/include/clang/Tooling/NodeIntrospection.h
    M clang/lib/Tooling/CMakeLists.txt
    R clang/lib/Tooling/DumpTool/APIData.h
    R clang/lib/Tooling/DumpTool/ASTSrcLocProcessor.cpp
    R clang/lib/Tooling/DumpTool/ASTSrcLocProcessor.h
    R clang/lib/Tooling/DumpTool/CMakeLists.txt
    R clang/lib/Tooling/DumpTool/ClangSrcLocDump.cpp
    R clang/lib/Tooling/DumpTool/generate_cxx_src_locs.py
    R clang/lib/Tooling/EmptyNodeIntrospection.inc.in
    R clang/lib/Tooling/NodeIntrospection.cpp
    M clang/unittests/CMakeLists.txt
    R clang/unittests/Introspection/CMakeLists.txt
    R clang/unittests/Introspection/IntrospectionTest.cpp
    M llvm/utils/gn/secondary/clang/lib/Tooling/BUILD.gn
    R llvm/utils/gn/secondary/clang/lib/Tooling/DumpTool/BUILD.gn
    M llvm/utils/gn/secondary/clang/unittests/BUILD.gn
    R llvm/utils/gn/secondary/clang/unittests/Introspection/BUILD.gn

  Log Message:
  -----------
  [clang-query] Remove support for srcloc output (#92442)

This functionality was added about three years ago, but has been in a
significantly broken state since it was added. It has begun to cause a
maintenance burden for work in Clang (largely due to the complexity of
having two levels of code generation involved), and the original author
is unable to help maintain it. Because it only worked under limited
circumstances and because of the maintenance burden, it is being
removed. If someone wishes to resurrect the functionality, they should
hopefully be able to do so from this one commit.

Fixes #82591



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list