[PATCH] D68185: [Diagnostics] Warn when class implements a copy constructor/copy assignment operator, but missing the copy assignment operator/copy constructor

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 12 13:25:30 PST 2019


dblaikie added a comment.

In D68185#1742809 <https://reviews.llvm.org/D68185#1742809>, @xbolva00 wrote:

> Ah, Clang has it under -Wdeprecated. https://clang.llvm.org/docs/DiagnosticsReference.html#wdeprecated.
>
> But combo -Wall -Wextra does not enable this warning, sadly.
>
> This should be extracted to -Wdeprecated-copy and put into -Wextra. Seems fine for you?


Sounds plausible to me - though I'd expect @rsmith should probably have a final say before that's submitted.


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

https://reviews.llvm.org/D68185





More information about the cfe-commits mailing list