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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
