[Lldb-commits] [PATCH] D137217: [LTO][COFF] Use bitcode file names in lto native object file names.

[Jez Ng via Phabricator via lldb-commits]

int3 added a comment.

I'm trying to add similar support to lld-macho, hence the question :)



================
Comment at: lld/COFF/LTO.cpp:183
+      [&](size_t task, const Twine &moduleName) {
+        buf[task].first = moduleName.str();
         return std::make_unique<CachedFileStream>(
----------------
Any reason why this doesn't instead store the module name in the `file_names[task]` vector that the cache callback uses? Are the task IDs not unique across each run of `ltoObj`, regardless of whether we end up using the cache or not?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137217/new/

https://reviews.llvm.org/D137217