[PATCH] D124420: [Serialization] Compress serialized macro expansion SLocEntries

2022-04-26 Thread Ilya Biryukov via Phabricator via cfe-commits
ilya-biryukov added inline comments. Comment at: clang/lib/Serialization/ASTWriter.cpp:2181 + // Common cases are covered by abbreviations. + unsigned Abbrev = [&]() -> unsigned { +if (!Expansion.isExpansionTokenRange()) // Token splits. NIT: ma

[PATCH] D124420: [Serialization] Compress serialized macro expansion SLocEntries

2022-04-25 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: ilya-biryukov. Herald added a project: All. sammccall requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Macro expansion SLocEntries are significant to PCH size (7-10% in my tests).