[all-commits] [llvm/llvm-project] acff42: [include-cleaner] Mark RecordDecls referenced in U...

kadir çetinkaya via All-commits all-commits at lists.llvm.org
Thu Aug 29 05:24:19 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: acff429191a27a164a0941346ce0c73e953d4638
      https://github.com/llvm/llvm-project/commit/acff429191a27a164a0941346ce0c73e953d4638
  Author: kadir çetinkaya <kadircet at google.com>
  Date:   2024-08-29 (Thu, 29 Aug 2024)

  Changed paths:
    M clang-tools-extra/include-cleaner/lib/WalkAST.cpp
    M clang-tools-extra/include-cleaner/unittests/WalkASTTest.cpp

  Log Message:
  -----------
  [include-cleaner] Mark RecordDecls referenced in UsingDecls as explicit (#106430)

We were reporting ambigious references from using declarations as user
can be depending on different overloads of a function just because they
are visible in the TU.
This doesn't apply to records, or primary templates as declaration being
referenced in such cases is unambigious, the ambiguity applies to
specializations though.

Hence this patch returns an explicit reference to record decls and
primary templates of those.



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