Page MenuHomePhabricator

[ELF] Disable ICF for synthetic sections
ClosedPublic

Authored by andrewng on Mar 27 2018, 6:03 AM.

Diff Detail

Event Timeline

andrewng created this revision.Mar 27 2018, 6:03 AM
ruiu added inline comments.Mar 27 2018, 6:20 AM
ELF/ICF.cpp
168

Please explain why. We don't merge synthetic sections because their Data member doesn't have a meaningful value.

170

Maybe it's better to write this as two if's because the boolean expression is a bit too complicated.

andrewng updated this revision to Diff 139914.Mar 27 2018, 6:49 AM

Updated to address review comments.

ruiu accepted this revision.Mar 27 2018, 6:51 AM

LGTM

Thanks!

ELF/ICF.cpp
173

You can use isa instead of dyn_cast.

This revision is now accepted and ready to land.Mar 27 2018, 6:51 AM
This revision was automatically updated to reflect the committed changes.