[PATCH] D151165: [ThinLTO] Make the cache key independent of the module identifier paths
Teresa Johnson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 31 12:38:08 PDT 2023
tejohnson added a comment.
In D151165#4538756 <https://reviews.llvm.org/D151165#4538756>, @tejohnson wrote:
> The module ID is an older concept that predates the module hash. We should probably remove that from the in-memory index completely, since it just takes up space and can lead to confusion about what values are stable and should be used. The numeric id is utilized in the Bitcode format for compactness, but I don't think we need it in memory anymore (quick scan of the codebase suggests not). Let me see if I can remove that once this issue is fixed.
D156730 <https://reviews.llvm.org/D156730>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151165/new/
https://reviews.llvm.org/D151165
More information about the llvm-commits
mailing list