[PATCH] D84244: [clang] Disable -Wsuggest-override for unittests/

Logan Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 21 16:48:06 PDT 2020


logan-5 added a comment.

In D84244#2165592 <https://reviews.llvm.org/D84244#2165592>, @dblaikie wrote:

> Looks OK (for future reference, this sort of stuff should've been cleaned up before enabling the flag so as to avoid this kind of hurry/breakage/etc)


Loud and clear. I honestly thought I had sifted through and dealt with all these before enabling it, but I was wrong. My mistake, I'm sorry about that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84244





More information about the cfe-commits mailing list