[PATCH] Don't dllimport/export destructor variants implemented by thunks.

Hans Wennborg hans at chromium.org
Tue May 27 14:47:50 PDT 2014


Hi rnk, nrieck,

MSVC doesn't export these functions, so trying to import them doesn't work.
Also, don't let any dll attributes on the CXXDestructorDecl influence the
thunk's linkage -- it should always be linkonce_odr.

This takes care of the FIXME's for this in Nico's tests.

http://reviews.llvm.org/D3930

Files:
  lib/CodeGen/CodeGenModule.cpp
  test/CodeGenCXX/dllimport-members.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3930.9849.patch
Type: text/x-patch
Size: 6698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140527/c73abb56/attachment.bin>


More information about the cfe-commits mailing list