[cfe-dev] Disabling GCC's -Woverloaded-virtual

Sam McCall via cfe-dev cfe-dev at lists.llvm.org
Fri Jun 26 02:33:41 PDT 2020


This warning isn't a GCC default, it's turned on in clang/CMakeLists.txt
(but not enabled for LLVM). This dates back to at least 2008 so I'm not
sure if the reasons still apply.

It has false positives by design that can't be silenced without (IMO)
making the code worse. Clang's version of the same warning doesn't have
this problem.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=20423

I've got https://reviews.llvm.org/D82617 which will make us enable it for
clang only, also happy to drop it entirely and let people who want it turn
it on.

Just wanted to check if anyone felt strongly about it, since I don't really
know who should review such a change.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20200626/46c5358e/attachment-0001.html>


More information about the cfe-dev mailing list