[Mips] Simplify the code a bit using early return
authorSimon Atanasyan <simon@atanasyan.com>
Fri, 24 Apr 2015 11:14:24 +0000 (11:14 +0000)
committerSimon Atanasyan <simon@atanasyan.com>
Fri, 24 Apr 2015 11:14:24 +0000 (11:14 +0000)
No functional changes.

llvm-svn: 235708

lld/lib/ReaderWriter/ELF/Mips/MipsRelocationPass.cpp

index 7818880..8947d93 100644 (file)
@@ -734,21 +734,13 @@ bool RelocationPass<ELFT>::isDynamic(const Atom *atom) const {
   const auto *da = dyn_cast<const DefinedAtom>(atom);
   if (da && da->dynamicExport() == DefinedAtom::dynamicExportAlways)
     return true;
-
-  const auto *sa = dyn_cast<SharedLibraryAtom>(atom);
-  if (sa)
+  if (isa<SharedLibraryAtom>(atom))
     return true;
-
-  if (_ctx.getOutputELFType() == ET_DYN) {
-    if (da && da->scope() != DefinedAtom::scopeTranslationUnit)
-      return true;
-
-    const auto *ua = dyn_cast<UndefinedAtom>(atom);
-    if (ua)
-      return true;
-  }
-
-  return false;
+  if (_ctx.getOutputELFType() != ET_DYN)
+    return false;
+  if (da && da->scope() != DefinedAtom::scopeTranslationUnit)
+    return true;
+  return isa<UndefinedAtom>(atom);
 }
 
 template <typename ELFT>