[PATCH] D72484: [clang-tidy] Fix check for Abseil internal namespace access
Gennadiy Rozental via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 13 09:20:48 PST 2020
rogeeff marked 2 inline comments as done.
rogeeff added inline comments.
================
Comment at: clang-tools-extra/clang-tidy/abseil/NoInternalDependenciesCheck.cpp:43-44
+
+ if (!LocAtFault.isValid())
+ return;
+
----------------
lebedev.ri wrote:
> Is there test coverage for this? When does/can this happen?
At the time when I wrote this internally the test cases in abseil-no-internal-dependencies.cpp were reproducing this failure. I'm not sure this is still the case. It is possible compiler was fixed since then.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72484/new/
https://reviews.llvm.org/D72484
More information about the cfe-commits
mailing list