[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