[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