This is an archive of the discontinued LLVM Phabricator instance.

[lld-macho][nfc] Remove TODO regarding addends
ClosedPublic

Authored by int3 on Feb 26 2021, 1:35 PM.

Details

Reviewers
thakis
Group Reviewers
Restricted Project
Commits
rGf083f652c3fd: [lld-macho][nfc] Remove TODO regarding addends
Summary

There was initially some concern around the correct handling of pcrel
section relocations with r_length != 2. But it looks like there are no such
relocations in practice -- x86_64's pcrel section relocs all have r_length == 2,
and ARM64 doesn't even have pcrel section relocs. So we can replace the TODO
with an assert.

Diff Detail

Event Timeline

int3 created this revision.Feb 26 2021, 1:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 26 2021, 1:35 PM
int3 requested review of this revision.Feb 26 2021, 1:35 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 26 2021, 1:35 PM
thakis accepted this revision.Feb 28 2021, 5:22 PM
This revision is now accepted and ready to land.Feb 28 2021, 5:22 PM
This revision was automatically updated to reflect the committed changes.