xbolva00 added a comment.

>> I've removed these warnings from the diagnostic groups -Wextra

But to remove all false positives, you need reports from users of -Wextra. 
Otherwise this would be dead code with no users.

I suggest to leave it in groups. Now you can ask folks to try this patch with 
kernel, Firefox, Chrome, XYZ and of course LLVM. Also you can try gcc’s 
testcases for this warning. If it is OK, I see no reasons to remove it from 
those groups.


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

Reply via email to