ChuanqiXu added a comment. In D137514#3912626 <https://reviews.llvm.org/D137514#3912626>, @dotnwat wrote:
> In D137514#3911046 <https://reviews.llvm.org/D137514#3911046>, @ChuanqiXu > wrote: > >> This is helpful. I feel it is OK to add the warning in the clang too. Are >> you interested in that? > > Yes, but unfortunately I'm very time constrained until the end of the year. > Is that something that would need to be added to this diff, or could be > separate? (fyi, brand new to clang development) Oh, I am not blocking this patch and this patch looks good to me and self-contained. That's just a simple suggestion. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137514/new/ https://reviews.llvm.org/D137514 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
