[PATCH] D72355: [clangd] Assert that the testcases in FindExplicitReferencesTest.All have no diagnostics
Nathan Ridge via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Jan 12 19:19:02 PST 2020
nridge added inline comments.
================
Comment at: clang-tools-extra/clangd/unittests/FindTargetTests.cpp:898
class $10^Foo {
- $11^Foo(int);
+ $11^Foo(int) {}
$12^Foo(): $13^Foo(111) {}
----------------
ilya-biryukov wrote:
> NIT: is this change redundant now? This was probably a warning, not an error.
You're right, reverted this change.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72355/new/
https://reviews.llvm.org/D72355
More information about the cfe-commits
mailing list