[PATCH] D113198: [lld-macho] Clear resolvedReads cache
Jez Ng via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 4 10:57:20 PDT 2021
int3 accepted this revision.
int3 added a comment.
This revision is now accepted and ready to land.
Thanks!
================
Comment at: lld/MachO/InputFiles.cpp:181
// would require altering many callers to track the state.
-static DenseMap<CachedHashStringRef, MemoryBufferRef> resolvedReads;
+DenseMap<CachedHashStringRef, MemoryBufferRef> macho::resolvedReads;
// Open a given file path and return it as a memory-mapped file.
----------------
unrelated nit: I'm thinking that `resolvedReads` is a bit of an odd name since we don't usually talk about reads being resolved, only paths... how about `cachedReads`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D113198/new/
https://reviews.llvm.org/D113198
More information about the llvm-commits
mailing list