[PATCH] Sema: Treat dllimport globals without explicit storage class as extern

Nico Rieck nico.rieck at gmail.com
Sat Mar 22 10:15:58 PDT 2014


On 21.03.2014 01:19, Richard Smith wrote:
> * goes on the right.
> [...]
> 'i' and 'e' should be capitalized.

That's what I get for copy & paste without looking closely... ;)

> You don't have any tests for the attribute being in the declarator chunks,
> rather than in the decl specifiers. Please add such tests.

Done, treating all three attribute placements the same, similar to other
attributes. I noticed that Clang and GCC handle such attributes
differently, i.e. GCC rejects something like
  int *__attribute__((visibility("hidden"))) v;

> Move the getAttr into the assert, so we trivially remove the whole thing in
> an NDEBUG build.

Done.

-Nico

-------------- next part --------------
A non-text attachment was scrubbed...
Name: dll-sema-global-dllimport-extern-v3.patch
Type: text/x-patch
Size: 8987 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140322/f620ea29/attachment.bin>


More information about the cfe-commits mailing list