[all-commits] [llvm/llvm-project] 8823a7: [NFC] [clang] add tests for merging of UsingShadow...

Matheus Izvekov via All-commits all-commits at lists.llvm.org
Thu Feb 1 11:18:51 PST 2024


  Branch: refs/heads/users/mizvekov/bug/clang-merge-usingshadowdecl
  Home:   https://github.com/llvm/llvm-project
  Commit: 8823a7ec248c7220395f53634467aa25f5007206
      https://github.com/llvm/llvm-project/commit/8823a7ec248c7220395f53634467aa25f5007206
  Author: Matheus Izvekov <mizvekov at gmail.com>
  Date:   2024-02-01 (Thu, 01 Feb 2024)

  Changed paths:
    A clang/test/Modules/GH80252.cppm
    A clang/test/Modules/cxx20-decls.cppm

  Log Message:
  -----------
  [NFC] [clang] add tests for merging of UsingShadowDecl


  Commit: f2e37f079cecdd95ba32912aaea739be68959f9e
      https://github.com/llvm/llvm-project/commit/f2e37f079cecdd95ba32912aaea739be68959f9e
  Author: Matheus Izvekov <mizvekov at gmail.com>
  Date:   2024-02-01 (Thu, 01 Feb 2024)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/lib/AST/ASTContext.cpp
    M clang/test/Modules/GH80252.cppm
    M clang/test/Modules/cxx20-decls.cppm

  Log Message:
  -----------
  [clang] fix merging of UsingShadowDecl

Previously, when deciding if two UsingShadowDecls where mergeable,
we would incorrectly only look for both pointing to the exact redecla
ration, whereas the correct thing is to look for declarations to the
same entity.

This problem has existed as far back as 2013, introduced in commit
fd8634a09de71.

This problem could manifest itself as ODR check false positives
when importing modules.

Fixes: #80252


Compare: https://github.com/llvm/llvm-project/compare/895c615a354d...f2e37f079cec


More information about the All-commits mailing list