[PATCH] D70165: clang-tidy: modernize-use-override new option AllowOverrideAndFinal

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 18 23:12:06 PST 2019


lebedev.ri added a comment.

In D70165#1750606 <https://reviews.llvm.org/D70165#1750606>, @mitchell-stellar wrote:

> Yes, there was a failing unit test that had to be fixed. I reverted the first commit and then committed a version that fixed the failing test.


I mean, the commit message (including differential link) was wrong.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70165





More information about the cfe-commits mailing list