MyDeveloperDay accepted this revision. MyDeveloperDay added a comment. This revision is now accepted and ready to land.
bool foo(int a, Bar) override; bool foo(int a, Bar) override; // comment bool foo(int a, Bar) final; bool foo(int a, Bar) final; // comment bool foo(a, Bar) final; // comment bool foo(a, Bar) final; Bar foo(a, Bar) final; // comment Bar foo(a, Bar) final; Bar foo(int a, Bar) final; // comment Bar foo(int a, Bar) final; Bar foo(a, Bar) noexcept; Bar foo(a, Bar) noexcept(false); Bar foo(int a, Bar) LLVM_OVERRIDE; Bar foo(a, Bar) LLVM_NOEXCEPT; Bar foo(a, Bar) LLVM_NOEXCEPT; Bar foo(a, Bar) const; bool foo(a, Bar) override; bool foo(a, Bar) const override; bool foo(a, Bar) noexcept override; bool foo(a, Bar) /*ABC*/ override; auto foo(a, Bar) OVERRIDE; [[nodiscard]] bool foo(a, Bar) final; [[nodiscard]] Bar foo(a, Bar) final; This is looking way more robust to me, LGTM.. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D107961/new/ https://reviews.llvm.org/D107961 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
