Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Assertion `Loc.isValid()' failed with recently added fuchsia-overloaded-operator check #35151

Closed
VReichelt opened this issue Jan 3, 2018 · 1 comment
Labels
bugzilla Issues migrated from bugzilla clang-tidy

Comments

@VReichelt
Copy link

Bugzilla Link 35803
Resolution FIXED
Resolved on Jan 03, 2018 14:19
Version unspecified
OS Linux
CC @juliehockett

Extended Description

Running the command
clang-tidy -checks='-*,fuchsia-overloaded-operator' bug.cc --
on the following valid code snippet

============================================================
void operator delete (void*, void*) throw();

triggers the following assertion:

clang-tidy: /llvm/tools/clang/tools/extra/clang-tidy/ClangTidyDiagnosticConsumer.cpp:193: clang::DiagnosticBuilder clang::tidy::ClangTidyContext::diag(llvm::StringRef, clang::SourceLocation, llvm::StringRef, clang::DiagnosticIDs::Level): Assertion `Loc.isValid()' failed.
Abort (core dumped)

This makes the switch -checks='*' completely unusable because this will
trigger with most system STL-headers as they use '#include ' at some
point from which the above snippet was extracted.

@juliehockett
Copy link
Mannequin

juliehockett mannequin commented Jan 3, 2018

Fixed in https://reviews.llvm.org/rCTE321762.

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla clang-tidy
Projects
None yet
Development

No branches or pull requests

1 participant