rsmith added a comment. If a patch is LGTM'd and some small change is requested at the same time, that typically means "LGTM once you make the following changes, which I trust you to make with no further pre-commit review". If you'd prefer more pre-commit review, of course, that's fine too.
In any case, this still LGTM, assuming it also includes the functional change itself (which is somehow missing from the latest revision here) =) http://reviews.llvm.org/D15095 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
