[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

Michael Benfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 15 11:49:27 PDT 2021


mbenfield added inline comments.


================
Comment at: clang/test/Sema/vector-gcc-compat.c:38
   v2i64 v2i64_a = (v2i64){0, 1};
-  v2i64 v2i64_r;
+  v2i64 v2i64_r; // expected-warning{{variable 'v2i64_r' set but not used}}
 
----------------
aeubanks wrote:
> this file isn't really testing this, I think this test should disable these warnings. I believe other tests do similar things, but it might be good grep around `clang/test` for `'Wno'` just to make sure
Makes sense, but is there a reason this test file uses `-Weverything` ?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D100581/new/

https://reviews.llvm.org/D100581



More information about the cfe-commits mailing list