This is an archive of the discontinued LLVM Phabricator instance.

[clang][AST] Add an AST matcher for deducedTemplateSpeializationType.
ClosedPublic

Authored by hokein on Feb 3 2020, 2:30 AM.

Details

Summary

misc-unused-using clang-tidy check needs this matcher to fix a false
positive of C++17 deduced class template types.

Diff Detail

Event Timeline

hokein created this revision.Feb 3 2020, 2:30 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2020, 2:30 AM
gribozavr2 accepted this revision.Feb 3 2020, 2:44 AM
gribozavr2 added inline comments.
clang/include/clang/ASTMatchers/ASTMatchers.h
6043

"in the declaration of the variable \c c"

This revision is now accepted and ready to land.Feb 3 2020, 2:44 AM
hokein updated this revision to Diff 242007.Feb 3 2020, 3:09 AM
hokein marked an inline comment as done.

address review comment.

This revision was automatically updated to reflect the committed changes.

Unit tests: pass. 62408 tests passed, 0 failed and 839 were skipped.

clang-tidy: pass.

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

Unit tests: pass. 62408 tests passed, 0 failed and 839 were skipped.

clang-tidy: pass.

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: diff.json, clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.