[PATCH] D53044: [ELF] Don't warn on undefined symbols if UnresolvedPolicy::Ignore is used

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 10 13:54:33 PDT 2018


ruiu added inline comments.


================
Comment at: ELF/Symbols.cpp:280-288
     Warn(": unable to order shared symbol: ");
-  else if (D && !D->Section)
-    Warn(": unable to order absolute symbol: ");
-  else if (D && isa<OutputSection>(D->Section))
-    Warn(": unable to order synthetic symbol: ");
-  else if (D && !D->Section->Repl->Live)
-    Warn(": unable to order discarded symbol: ");
+  } else if (D) {
+    if (!D->Section)
+      Warn(": unable to order absolute symbol: ");
+    else if (isa<OutputSection>(D->Section))
+      Warn(": unable to order synthetic symbol: ");
+    else if (!D->Section->Repl->Live)
----------------
Please do not change this -- this code was written in the way it is to reduce indentation depth.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D53044





More information about the llvm-commits mailing list