[PATCH] D85153: [lld-macho] Add .tbd support for frameworks

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 08:51:39 PDT 2020


compnerd accepted this revision.
compnerd added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lld/MachO/Driver.cpp:87
+static Optional<std::string>
+findWithAnyExtension(StringRef base, ArrayRef<StringRef> extensions) {
+  for (StringRef ext : extensions) {
----------------
The name makes it seem like it will match `base.*`.  What do you think of `findWithAllowedExtensions`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85153



More information about the llvm-commits mailing list