[PATCH] D84206: [llvm-libtool-darwin] Add support for -filelist option

[Sameer Arora via Phabricator via llvm-commits]

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


================
Comment at: llvm/tools/llvm-libtool-darwin/llvm-libtool-darwin.cpp:78
+      sys::path::append(Path, Line);
+    InputFiles.push_back(Twine(Path).str());
+  }
----------------
smeenai wrote:
> Nit: SmallString has an `operator std::string`, so you should be able to convert directly without the intermediate Twine. I'm not sure if just `InputFiles.push_back(Path)` will compile, but a cast should definitely trigger the conversion, e.g. `InputFiles.push_back(static_cast<std::string>(Path))`.
Yup, casting works. Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84206