[PATCH] D84858: [llvm-libtool-darwin] Refactor ArchiveWriter

[Sameer Arora via Phabricator via llvm-commits]

sameerarora101 marked an inline comment as done.
sameerarora101 added inline comments.


================
Comment at: llvm/lib/Object/ArchiveWriter.cpp:639
+                                     Deterministic, Thin))
+    return E;
 
----------------
smeenai wrote:
> I believe you need to explicitly call `Temp->discard()` in this path, otherwise you'll trip [this assert](https://github.com/llvm/llvm-project/blob/4aaa97700377a811dbcaf88a81ec068a5b517a4d/llvm/lib/Support/Path.cpp#L1171). Note that `discard()` itself can return an error, in which case you probably want to do `joinErrors`?
Oh makes sense, thanks. Added it now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84858