[PATCH] D102186: [clang][c++20] Fix false warning for unused private fields when a class has only defaulted comparison operators.

[Hans Wennborg via Phabricator via cfe-commits]

hans added inline comments.


================
Comment at: clang/test/SemaCXX/warn-unused-private-field.cpp:7
+
+class EqDefaultCompare {
+  int unused_;
----------------
The test case feels a little strange to me, I mean the member variable is still 
pretty unused even though I suppose the comparison function technically uses 
it.. At least the test case in the bug report had a constructor which set the 
value. Perhaps with that it makes more sense.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102186

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits