owenpan added a comment. In D60374#1457693 <https://reviews.llvm.org/D60374#1457693>, @MyDeveloperDay wrote:
> LGTM , if you also think the test will help show the use case then please add > it, otherwise this revision notes might be information enough Thanks! I just got the alternative patch ready. Should I discard it and just commit this one? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60374/new/ https://reviews.llvm.org/D60374 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
