Fixed bug for archives

Author: JDevlieghere

lib/src/BitcodeRetriever.cpp

@@ -10,6 +10,7 @@
 #include "llvm/Object/MachOUniversal.h"
 
 #include <algorithm>
+#include <iostream>
 #include <memory>
 #include <string>
 
@@ -55,9 +56,8 @@ std::vector<std::unique_ptr<BitcodeContainer>> BitcodeRetriever::GetBitcodeConta
           bitcodeContainers.push_back(std::move(container));
         }
         continue;
-      } else {
-        // Calling operator bool() is not sufficient.
-        machOObject.takeError();
+      } else if (auto e = machOObject.takeError()) {
+        llvm::consumeError(std::move(e));
       }
 
       Expected<std::unique_ptr<Archive>> archive = object.getAsArchive();
@@ -67,9 +67,8 @@ std::vector<std::unique_ptr<BitcodeContainer>> BitcodeRetriever::GetBitcodeConta
         bitcodeContainers.reserve(bitcodeContainers.size() + containers.size());
         std::move(std::begin(containers), std::end(containers), std::back_inserter(bitcodeContainers));
         continue;
-      } else {
-        // Calling operator bool() is not sufficient.
-        archive.takeError();
+      } else if (auto e = archive.takeError()) {
+        llvm::consumeError(std::move(e));
       }
 
       throw EbcError("Unrecognized MachO universal binary");
@@ -99,6 +98,10 @@ std::vector<std::unique_ptr<BitcodeContainer>> BitcodeRetriever::GetBitcodeConta
   Error err;
   auto children = archive.children(err);
 
+  if (err) {
+    throw EbcError("Couldn't get children from archive " + archive.getFileName().str());
+  }
+
   auto bitcodeContainers = std::vector<std::unique_ptr<BitcodeContainer>>();
   for (const auto &child : children) {
     auto childOrErr = child.getAsBinary();

tool/ebcutil.cpp

@@ -89,7 +89,7 @@ int main(int argc, char* argv[]) {
     TCLAP::ValueArg<std::string> prefixArg("p", "prefix", "Prefix for bitcode files", false, "", "string");
     cmd.add(prefixArg);
 
-    TCLAP::UnlabeledValueArg<std::string> fileArg("File", "Library or object file", false, "", "file");
+    TCLAP::UnlabeledValueArg<std::string> fileArg("File", "Library or object file", true, "", "file");
     cmd.add(fileArg);
 
     cmd.parse(argc, argv);