[PATCH] D70931: [MS] Emit exported complete/vbase destructors

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 3 02:29:49 PST 2019


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
+  // dllexported inlines are being exported.
----------------
The grammar looks funny here: "when the class was dllexported inlines are being exported".


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70931/new/

https://reviews.llvm.org/D70931





More information about the cfe-commits mailing list