[PATCH] D150494: [clang][modules] NFC: Only sort interesting identifiers

2023-05-15 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGf60cc473b82b: [clang][modules] NFC: Only sort interesting identifiers (authored by jansvoboda11). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D150494: [clang][modules] NFC: Only sort interesting identifiers

2023-05-15 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 added inline comments. Comment at: clang/lib/Serialization/ASTWriter.cpp:3646 // file. SmallVector IIs; for (const auto : PP.getIdentifierTable()) ributzka wrote: > Would it make sense to reduce the size of the SmallVector too? That

[PATCH] D150494: [clang][modules] NFC: Only sort interesting identifiers

2023-05-15 Thread Juergen Ributzka via Phabricator via cfe-commits
ributzka added inline comments. Comment at: clang/lib/Serialization/ASTWriter.cpp:3646 // file. SmallVector IIs; for (const auto : PP.getIdentifierTable()) Would it make sense to reduce the size of the SmallVector too? Repository: rG LLVM

[PATCH] D150494: [clang][modules] NFC: Only sort interesting identifiers

2023-05-12 Thread Jan Svoboda via Phabricator via cfe-commits
jansvoboda11 created this revision. jansvoboda11 added reviewers: benlangmuir, Bigcheese. Herald added a subscriber: ributzka. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. In 9c254184