r260548 - Revert "Revert r260388 "[MS ABI] Never reference dllimport'd vtables""

2016-02-11 Thread David Majnemer via cfe-commits
Author: majnemer Date: Thu Feb 11 11:49:28 2016 New Revision: 260548 URL: http://llvm.org/viewvc/llvm-project?rev=260548&view=rev Log: Revert "Revert r260388 "[MS ABI] Never reference dllimport'd vtables"" This reverts commit r260449. We would supress our emission

Re: r260388 - [MS ABI] Never reference dllimport'd vtables

2016-02-10 Thread Hans Wennborg via cfe-commits
On Wed, Feb 10, 2016 at 9:40 AM, David Majnemer via cfe-commits wrote: > Author: majnemer > Date: Wed Feb 10 11:40:47 2016 > New Revision: 260388 > > URL: http://llvm.org/viewvc/llvm-project?rev=260388&view=rev > Log: > [MS ABI] Never reference dllimport'd vtables > > Referencing a dllimported vta

r260449 - Revert r260388 "[MS ABI] Never reference dllimport'd vtables"

2016-02-10 Thread Hans Wennborg via cfe-commits
Author: hans Date: Wed Feb 10 16:18:37 2016 New Revision: 260449 URL: http://llvm.org/viewvc/llvm-project?rev=260449&view=rev Log: Revert r260388 "[MS ABI] Never reference dllimport'd vtables" This caused the compiler to fail with "invalid linkage type for glob

r260388 - [MS ABI] Never reference dllimport'd vtables

2016-02-10 Thread David Majnemer via cfe-commits
Author: majnemer Date: Wed Feb 10 11:40:47 2016 New Revision: 260388 URL: http://llvm.org/viewvc/llvm-project?rev=260388&view=rev Log: [MS ABI] Never reference dllimport'd vtables Referencing a dllimported vtable is impossible in a constexpr constructor. It would be friendlier to C++ programmers