[cfe-commits] [Request for approval] Wrong extern inside no braces linkage specification declaration

[Abramo Bagnara]

Currently clang marks these two declarations

extern "C" int a;
extern "C" void foo();

as extern in StorageClassAsWritten, but the inner declarations does not
have any storage class specified (and C++ standard does not allow any).

The attached patch wrt r102259 fixes the problem (and do not introduce
any regressions).

This patch has already been sent to cfe-dev, currently without pending
objections but without explicit approval.

http://lists.cs.uiuc.edu/pipermail/cfe-dev/2010-April/008773.html

Having just obtained write access, is it ok if I commit it in?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: linkage-extern.patch
Type: text/x-patch
Size: 4953 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20100428/0a0785dd/attachment.bin>