[PATCH] D43241: [ELF] - Do not crash with --emit-relocs and --icf=all together.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 10:39:41 PST 2018


ruiu added a comment.

`replace` is not designed to be smart, and adding new code there seems semantically incorrect, as replacing some section doesn't necessarily mean that its dependent sections should be marked as dead.

You could instead add code to ICF.cpp.


https://reviews.llvm.org/D43241





More information about the llvm-commits mailing list