[cfe-dev] Proposal to fix issue 7715

Manuel Klimek klimek at google.com
Tue Sep 14 07:39:44 PDT 2010


Hi,

I came up with a solution to http://llvm.org/bugs/show_bug.cgi?id=7715
(clang++ gives wrong error about default argument in kdatetime.h), as
talked about on IRC. It was a little more complicated than
anticipated, as we need to keep the stack of scopes correct.

I would like to get some feedback on the general direction /
implementation of this patch. If we agree this is the way to go, I'd
comment it more thoroughly and add some more regression tests. I tried
to keep the diff as small as possible, but my gut feeling would also
lead me to put more stuff into the classes as opposed to having
everything as methods on Parser. Not sure though.

Cheers,
/Manuel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-7715.patch
Type: text/x-patch
Size: 24272 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20100914/44476071/attachment.bin>


More information about the cfe-dev mailing list