[clang] Warn about virtual methods in `final` classes (PR #131188)

[via cfe-commits]

================
@@ -0,0 +1,29 @@
+// RUN: %clang_cc1 -fsyntax-only -verify -Wunnecessary-virtual-specifier %s
+
+struct Foo final {
+  Foo() = default;
+  virtual ~Foo() = default;                      // expected-warning {{virtual method}}
+  virtual Foo& operator=(Foo& other) = default;  // expected-warning {{virtual method}}
+  virtual Foo& operator=(Foo&& other) = default; // expected-warning {{virtual method}}
+  void f();
+  virtual void f(int);                           // expected-warning {{virtual method}}
+  int g(int x) { return x; };
+  virtual int g(bool);                           // expected-warning {{virtual method}}
+  static int s();
+};
+
+struct BarBase {
+  virtual ~BarBase() = delete;
+  virtual void virt() {}
+  virtual int virt(int);
+  int nonvirt();
+};
+
+struct Bar final : BarBase {
+  ~Bar() override = delete;
+  void virt() override {};
+  // `virtual ... override;` is a common pattern, so don't warn
+  virtual int virt(int) override;
+  virtual int virt(bool);                        // expected-warning {{virtual method}}
----------------
cor3ntin wrote:

Oh, nvm, I misread the added test. I was asking the same thing as @zmodem https://github.com/llvm/llvm-project/pull/131188/files#r1995849109

https://github.com/llvm/llvm-project/pull/131188