[PATCH] D25433: [ELF] Don't crash if relocation target section is discarded

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 10 15:34:48 PDT 2016


ruiu added inline comments.


================
Comment at: ELF/Symbols.cpp:57
+    if (!SC->Live) {
+      warn("relocation refers to discarded section");
+      return D.Value;
----------------
Please include a filename and symbol name in the error message.


================
Comment at: ELF/Symbols.cpp:58
+      warn("relocation refers to discarded section");
+      return D.Value;
+    }
----------------
Isn't it better to return 0 instead of D.Value?


Repository:
  rL LLVM

https://reviews.llvm.org/D25433





More information about the llvm-commits mailing list