[PATCH] D134784: [IR] Don't allow DLL storage-class and local linkage

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

In D134784#3822333 <https://reviews.llvm.org/D134784#3822333>, @MaskRay wrote:

> `__declspec(dllexport) const int foo = 10;` foo with internal linkage is accepted.

I believe in this case MSVC will make the symbol external, so it's not actually internal. The dll attributes tend to imply `extern` in many cases.


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

https://reviews.llvm.org/D134784