[cfe-commits] [PATCH][MS][Review request] - Microsoft C anonymous struct, second try

Francois Pichet pichet2000 at gmail.com
Sun Oct 24 18:39:12 PDT 2010


Hi

This is an updated patch for the Microsoft C anonymous struct
extensions. (PR 6829)
MSDN documentation: http://msdn.microsoft.com/en-us/library/z2cx9y4f.aspx

My first patch didn't deal with anonymous struct defined via typedef.
for example:
typedef struct { int a} A;
struct B {A; int b;};

I struggled and I finally decided to deal that case by inserting an
implicit RecordDecl to make the handling uniform. (To have
isTransparentContext return true)
An alternative solution would be to have isTransparentContext() deal
with a sole typedef name as a transparent context. But I found that
solution intrusive for such an obscure MS C extension.

Please review.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ms-c-anonymous-struct.patch
Type: application/octet-stream
Size: 21042 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20101024/c1e85825/attachment.obj>


More information about the cfe-commits mailing list