elsteveogrande added a comment. Looks like I have the "accept" ability in this repo, wasn't sure since I just registered for this recently :)
I think since the tests look good, it's a nice and elegant 1-line change, per the reviewer's (@rsmith 's) comments, this should be ok to go. This would help greatly on a project I'm working on, which has reams of auto-generated headers, so naturally I'm antsy to get this in trunk :) @wristow, if this still lands cleanly against trunk (I see that this has been around a while) and test still pass, think this can be committed soon-ish? Thanks! https://reviews.llvm.org/D19815 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
