[PATCH] D124422: [Serialization] Improve encoding of small SourceRanges.
Sam McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 25 17:12:27 PDT 2022
sammccall added a comment.
Bad news: this isn't ready, there are places where ASTWriter calls AddSourceLocation() twice and ASTReader calls ReadSourceRange().
After this change, those are no longer compatible so we have to make them match, and have to audit them all.
Good news, there's a bunch of potential places where we can replace AddSourceLocation() pairs with AddSourceRange for further savings.
Maybe even replacing other "chains" of source locations with delta-encoding, though I might be getting carried away...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124422/new/
https://reviews.llvm.org/D124422
More information about the cfe-commits
mailing list