[PATCH] D94461: [llvm-ifs] Add option to use InterfaceStub library

[Puyan Lotfi via Phabricator via llvm-commits]

plotfi added inline comments.


================
Comment at: llvm/tools/llvm-ifs/llvm-ifs.cpp:58
 
+static cl::opt<bool> UseInterfaceStub(
+    "use-interfacestub",
----------------
Put a comment here to the effect that the InterfaceStubs backend is newer than "Clang InterfaceStubs" which originally used the yaml2obj backend. Eventually I assume we want to flip the default for this too. Can also you explicitly set the default here to make that clear? 


================
Comment at: llvm/tools/llvm-ifs/llvm-ifs.cpp:552
 
+  if (UseInterfaceStub && (Action == "write-bin")) {
+    elfabi::ELFStub ElfStub;
----------------
What happens if UseInterfaceStub  == true but Action is not "write-bin" ? Shouldn't it generate a merged llvm-elftapi text file or something like that or at least error out? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94461