This revision was automatically updated to reflect the committed changes.
Closed by commit rG705a6aef3502: [MS] Emit exported complete/vbase destructors
(authored by rnk).
Changed prior to commit:
https://reviews.llvm.org/D70931?vs=231796&id=231998#toc
Repository:
rG LLVM Github Monorepo
CH
rnk marked an inline comment as done.
rnk added inline comments.
Comment at: clang/lib/CodeGen/MicrosoftCXXABI.cpp:1349
+ // the base dtor is emitted.
+ // FIXME: To match MSVC, this should only be done when the class was
+ // dllexported inlines are being exported.
--
hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.
Nice, thanks!
Comment at: clang/lib/CodeGen/MicrosoftCXXABI.cpp:1349
+ // the base dtor is emitted.
+ // FIXME: To match MSVC, this should only be done when the class was
+ //
rnk created this revision.
rnk added a reviewer: hans.
Herald added a project: clang.
Fixes PR44205
I checked, and deleting destructors are not affected.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D70931
Files:
clang/lib/CodeGen/MicrosoftCXXABI.cpp
clang/test/CodeGenCX