[PATCH] D34714: [MS] Don't statically initialize dllimport member function pointers

David Majnemer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 27 15:44:53 PDT 2017


majnemer created this revision.

r306137 made dllimport pointers to member functions non-constant. This
is correct because a load must be executed to resolve any dllimported
data. However, r306137 did not account for the use of dllimport member
function pointers used as template arguments.

This change piggie-backs almost entirely on Reid's r306137, I just added
the template instantiation fix.

This fixes PR33570.


https://reviews.llvm.org/D34714

Files:
  lib/AST/ExprConstant.cpp
  lib/Sema/SemaTemplate.cpp
  test/CodeGenCXX/dllimport-memptr-global.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D34714.104289.patch
Type: text/x-patch
Size: 7644 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170627/964eb87a/attachment-0001.bin>


More information about the cfe-commits mailing list