llvm-project
[modules] Handle friend function that was a definition but became only a declaration during AST deserialization
#132214
Merged

[modules] Handle friend function that was a definition but became only a declaration during AST deserialization #132214

dmpolukhin
dmpolukhin [clang] Fix for regression #130917
91e057bf
dmpolukhin dmpolukhin requested a review from mizvekov mizvekov 266 days ago
dmpolukhin dmpolukhin requested a review from ChuanqiXu9 ChuanqiXu9 266 days ago
llvmbot llvmbot added clang
llvmbot llvmbot added clang:frontend
llvmbot
mizvekov
mizvekov commented on 2025-03-20
dmpolukhin Add test case with C++ modules
adf6dfde
dmpolukhin Add flag to FunctionDecl
a564da23
llvmbot llvmbot added clang:modules
github-actions
dmpolukhin Fix formatting
2b42b4d2
dmpolukhin Add missing initialization
8d5ecfd1
ChuanqiXu9
dmpolukhin dmpolukhin changed the title [RFC][clang] Fix for regression #130917 [RFC][clang] Handle friend function that was a definition but became only a declaration during AST deserialization 260 days ago
cor3ntin cor3ntin requested a review from AaronBallman AaronBallman 260 days ago
dmpolukhin Move ThisDeclarationWasADefinition to ExternalASTSource
df9fd8b4
ChuanqiXu9
ChuanqiXu9 commented on 2025-03-28
ChuanqiXu9
dmpolukhin Rename + add only friend functions to the map
d7234be3
dmpolukhin dmpolukhin changed the title [RFC][clang] Handle friend function that was a definition but became only a declaration during AST deserialization [modules] Handle friend function that was a definition but became only a declaration during AST deserialization 258 days ago
dmpolukhin dmpolukhin requested a review from ChuanqiXu9 ChuanqiXu9 255 days ago
dmpolukhin Move ThisDeclarationWasADefinition to separate map
94efa552
ChuanqiXu9
ChuanqiXu9 commented on 2025-04-02
dmpolukhin Replace map with set
a24e8db8
dmpolukhin
dmpolukhin commented on 2025-04-02
dmpolukhin Fix formatting
b8870506
ChuanqiXu9
ChuanqiXu9 commented on 2025-04-02
dmpolukhin Add corresponding changes in MultiplexExternalSemaSource
0a99d891
dmpolukhin
ChuanqiXu9
ChuanqiXu9 approved these changes on 2025-04-03
dmpolukhin dmpolukhin merged e1aaee7e into main 252 days ago
llvm-ci

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone