[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