[cfe-commits] [PATCH] Fix ParenListExpr with null type
Manuel Klimek
klimek at google.com
Tue Jun 14 18:06:22 PDT 2011
The attached patch
- fixes the introduction of null types for ParenListExpr's that end up
in the AST for explicit initializers by making the constructor of
ParenListExpr take a type (as suggested by dgregor on irc)
- gets rid of some code I assume is dead (tested by running the tests
and by running it over all of our internal C++ code without hitting
any of those asserts - and by my inability to come up with an example
that hits that code path (which admittedly doesn't mean a lot))
- tested that the other in-use cases that create ParenListExpr's
without a type don't lead to them being in the AST in the end
Rietveld link:
http://codereview.appspot.com/4584059/
Cheers,
/Manuel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: issue4584059_1.diff
Type: text/x-patch
Size: 8083 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110614/743b7c69/attachment.bin>
More information about the cfe-commits
mailing list