MyDeveloperDay added a comment. In D60374#1457705 <https://reviews.llvm.org/D60374#1457705>, @owenpan wrote:
> 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? Maybe just update this revision with the new diff. 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
