[PATCH] D46598: Add support for LTO plugin option obj-path

[Rumeet Dhindsa via Phabricator via llvm-commits]

rdhindsa added inline comments.


================
Comment at: lld/ELF/LTO.cpp:283
+    if (Config->LTOObjPath.empty())
       saveBuffer(Buff[0], Config->OutputFile + ".lto.o");
+    else
----------------
pcc wrote:
> I wouldn't save the buffer at all if `obj-path` is not passed.
Wouldn't we want to store the generated file, if obj-path is not passed? Since we know that Buff[0] is not empty, aren't we supposed to store the content?  In that case, is obj-path a requirement for thinlto-index-only?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D46598