mbenfield added a comment. In D100581#2730557 <https://reviews.llvm.org/D100581#2730557>, @nickdesaulniers wrote:
> I see lots of instances from the kernel that look like this when reduced: > But adding a new warning flags to a handful of existing tests' RUN lines is > unusual. These tests use `-Wall` for whatever reason, and they assign but don't use several variables, so we'll get warnings on them unless I add `-Wno-unused-but-set-variable`. Other alternatives: 1. add an expected-warning; 2. use a pragma to disable the warning; 3. put an `unused` attribute on the variables in question; 4. introduce an artificial use of these variables. I previously used item 1, but somebody earlier in an earlier comment requested that instead I add `-Wno-unused-but-set-variable` instead. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100581/new/ https://reviews.llvm.org/D100581 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
