alexfh added a comment. In D105890#2873969 <https://reviews.llvm.org/D105890#2873969>, @thakis wrote:
> I think this is incorrect. See D105892 <https://reviews.llvm.org/D105892> and > https://reviews.llvm.org/D104915#2873851 . Don't blindly land changes to > suppress warnings – the warnings exist to tell us something :) Indeed, thanks for pointing this out! On the other hand, it may be worth adding this warning with -Werror to the default set or at least enable it when running pre-merge checks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105890/new/ https://reviews.llvm.org/D105890 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
