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

[ben via Phabricator via llvm-commits]

bd1976llvm added a comment.

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

> For Clang, it seems that MSVC allows dllspec on a local linkage variable, so that's why we have to add it to CodeGen instead of rejecting it in Sema.

Can you give an example? MSVC certainly rejects simple cases e.g. https://godbolt.org/z/bG3M7oj5W. I am worried that I have missed something...


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

https://reviews.llvm.org/D134784