gribozavr2 added a comment.
What is this diff based on? On the left I see, for example,
NamespaceNameSpecifier, which is not in the repository yet.
================
Comment at: clang/lib/Tooling/Syntax/BuildTree.cpp:777
// FIXME: Support Microsoft's __super
- return new (allocator()) syntax::UnknownNameSpecifier;
+ assert(false && "We don't yet treat the __super specifier");
}
----------------
s/treat/support/
Also, use llvm::report_fatal_error instead? assert is not supposed to ever
trigger.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D84348/new/
https://reviews.llvm.org/D84348
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
