[PATCH] D138648: [include-cleaner] Make Symbol (and Macro) hashable.

2022-11-24 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG0cb2dd5322f4: [include-cleaner] Make Symbol (and Macro) hashable. (authored by sammccall). Changed prior to commit: https://reviews.llvm.org/D138648?vs=477723&id=477768#toc Repository: rG LLVM Github

[PATCH] D138648: [include-cleaner] Make Symbol (and Macro) hashable.

2022-11-24 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks, lgtm! Comment at: clang-tools-extra/include-cleaner/include/clang-include-cleaner/Types.h:147 + using Outer = clang::include_cleaner::Symbol; + constexpr stati

[PATCH] D138648: [include-cleaner] Make Symbol (and Macro) hashable.

2022-11-24 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. For now, we decided not to add operator< or handle other variants. (If