This is an archive of the discontinued LLVM Phabricator instance.

Update SanitizerCoverage doc regarding the issue with pc-table and gc-sections.
ClosedPublic

Authored by Dor1s on Jan 30 2019, 2:24 PM.

Diff Detail

Repository
rC Clang

Event Timeline

Dor1s created this revision.Jan 30 2019, 2:24 PM
Dor1s added a comment.Jan 30 2019, 2:24 PM

I'm not sure if I wrote down the information correctly, PTAL :)

morehouse added inline comments.Jan 31 2019, 10:29 AM
docs/SanitizerCoverage.rst
149

Maybe add "for linkers other than LLD". Dead stripping should work 100% for LLD now.

Dor1s updated this revision to Diff 184766.Feb 1 2019, 9:02 AM

Clarify that only linkers other than LLD are affected.

Herald added a project: Restricted Project. · View Herald TranscriptFeb 1 2019, 9:02 AM
Dor1s marked 2 inline comments as done.Feb 1 2019, 9:02 AM
Dor1s added inline comments.
docs/SanitizerCoverage.rst
149

Thanks, Matt! Done.

This revision is now accepted and ready to land.Feb 1 2019, 9:04 AM
This revision was automatically updated to reflect the committed changes.
Dor1s marked an inline comment as done.