[clang-tools-extra] [clang-tidy] Add option to keep virtual in 'modernize-use-override' (PR #144916)

Baranov Victor via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 19 09:51:49 PDT 2025


================
@@ -0,0 +1,25 @@
+// RUN: %check_clang_tidy %s modernize-use-override %t -- \
+// RUN:   -config="{CheckOptions: {modernize-use-override.AllowVirtual: true}}"
+
+struct Base {
+  virtual ~Base();
+  virtual void a();
+  virtual void b();
+  virtual void c();
+};
+
+struct Derived : public Base {
+  virtual ~Derived() override;
+
+  virtual void a() override;
+  // CHECK-MESSAGES-NOT: warning:
+  // CHECK-FIXES: {{^}}  virtual void a() override;
----------------
vbvictor wrote:

You should just write nothing. By default, tests will fail if there will be a message

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


More information about the cfe-commits mailing list