Remove excessive allowRemainingUndefines() check.

allowRemainingUndefines() is already checked in Resolver::resolve(), so we
don't need to check it again after returning from it. It's actually not only
superfluous but buggy because a failure of resolve() does not always mean that
there is a remaining undefines.

llvm-svn: 192423
diff --git a/lld/lib/Driver/Driver.cpp b/lld/lib/Driver/Driver.cpp
index 6fdb343..05532e8 100644
--- a/lld/lib/Driver/Driver.cpp
+++ b/lld/lib/Driver/Driver.cpp
@@ -107,10 +107,8 @@
   // Do core linking.
   ScopedTask resolveTask(getDefaultDomain(), "Resolve");
   Resolver resolver(context);
-  if (!resolver.resolve()) {
-    if (!context.allowRemainingUndefines())
-      return false;
-  }
+  if (!resolver.resolve())
+    return false;
   MutableFile &merged = resolver.resultFile();
   resolveTask.end();