Looks great, with one tweak. ================ Comment at: lib/Parse/ParseDecl.cpp:5646-5649 @@ +5645,6 @@ + + // Nothing after the brackets has been parsed as part of the Declarator, so + // no correction is needed. + if (OldEndLoc == D.getLocEnd()) + return; + ---------------- This looks unreachable. If nothing after the brackets has been parsed, then there can't be an identifier, so we should have bailed out already. Remove?
http://reviews.llvm.org/D2712 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
