[all-commits] [llvm/llvm-project] 721297: [clangd] Always desugar type aliases in hover

kadir ├žetinkaya via All-commits all-commits at lists.llvm.org
Wed Jun 15 07:13:01 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7212977fbb41bcffaf28a33a70c2df2b40492524
      https://github.com/llvm/llvm-project/commit/7212977fbb41bcffaf28a33a70c2df2b40492524
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2022-06-15 (Wed, 15 Jun 2022)

  Changed paths:
    M clang-tools-extra/clangd/Hover.cpp
    M clang-tools-extra/clangd/unittests/HoverTests.cpp

  Log Message:
  -----------
  [clangd] Always desugar type aliases in hover

The alias itself is already included in the definition section of the
hover (it's printed as spelled in source code). So it doesn't provide any value
when we print the aliases as-is.
Fixes https://github.com/clangd/clangd/issues/1134.

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


  Commit: 04735302814e4df1cf37caeff3406fa5523901eb
      https://github.com/llvm/llvm-project/commit/04735302814e4df1cf37caeff3406fa5523901eb
  Author: Kadir Cetinkaya <kadircet at google.com>
  Date:   2022-06-15 (Wed, 15 Jun 2022)

  Changed paths:
    M clang-tools-extra/clangd/Hover.cpp

  Log Message:
  -----------
  [clangd][NFC] Use the existing ASTContext from scope


Compare: https://github.com/llvm/llvm-project/compare/8f2ba3633646...04735302814e


More information about the All-commits mailing list