aaron.ballman added a comment. In D111400#3400011 <https://reviews.llvm.org/D111400#3400011>, @hubert.reinterpretcast wrote:
> Confirming LGTM with minor comments. > > In D111400#3399929 <https://reviews.llvm.org/D111400#3399929>, @cor3ntin > wrote: > >> - We may still miss a compatibility warning for non-literal in c++23 mode? > > My understanding is that Aaron and I both want the warning added (but are > okay with having this patch landed first). +1 to this Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D111400/new/ https://reviews.llvm.org/D111400 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits