Do not return early.
authorRui Ueyama <ruiu@google.com>
Sat, 2 Apr 2016 18:52:23 +0000 (18:52 +0000)
committerRui Ueyama <ruiu@google.com>
Sat, 2 Apr 2016 18:52:23 +0000 (18:52 +0000)
Early return in this context is a bit confusing, so avoid doing it.

llvm-svn: 265236

lld/ELF/Driver.cpp

index a825d6b..71bcc8c 100644 (file)
@@ -174,17 +174,16 @@ static void checkOptions(opt::InputArgList &Args) {
   if (Config->Pie && Config->Shared)
     error("-shared and -pie may not be used together");
 
-  if (!Config->Relocatable)
-    return;
-
-  if (Config->Shared)
-    error("-r and -shared may not be used together");
-  if (Config->GcSections)
-    error("-r and --gc-sections may not be used together");
-  if (Config->ICF)
-    error("-r and --icf may not be used together");
-  if (Config->Pie)
-    error("-r and -pie may not be used together");
+  if (Config->Relocatable) {
+    if (Config->Shared)
+      error("-r and -shared may not be used together");
+    if (Config->GcSections)
+      error("-r and --gc-sections may not be used together");
+    if (Config->ICF)
+      error("-r and --icf may not be used together");
+    if (Config->Pie)
+      error("-r and -pie may not be used together");
+  }
 }
 
 static StringRef