[PATCH] D84126: Enable -Wsuggest-override in the LLVM build

Erich Keane via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 22 07:47:57 PDT 2020


erichkeane added a comment.

Ive got a bit a of a problem here: This flag (suggest-override) on GCC 8.3 (and others) warns on missing-override EVEN WHEN the function is marked 'final'.  This was not fixed until GCC 9.2: https://godbolt.org/z/55KeM3

The result is my build gave a few thousand diagnostics for this.  I'd suggest disabling this unless the GCC version is >9.1.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84126





More information about the llvm-commits mailing list