================
Comment at: lib/Sema/SemaDecl.cpp:9108
@@ -9107,1 +9107,3 @@
+ // Static locals inherit dll attributes from their function.
+ if (VD->isStaticLocal()) {
----------------
Reid Kleckner wrote:
> Should this go before checkAttributesAfterMerging? Consider this obnoxious
> test case:
> int f();
> inline __declspec(dllexport) int g() {
> static __declspec(dllimport) int x = f();
> return x;
> }
Local variables cannot have explicit dll attributes. They can only get them by
inheriting from the function they're part of, so calling
checkAttributesAfterMerging to check up on the attribute we just added seems
redundant.
http://reviews.llvm.org/D4136
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
