[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