This is an archive of the discontinued LLVM Phabricator instance.

Fixes handling logic for i386/ELF GOTPC relocation
ClosedPublic

Authored by jain98 on Jan 13 2023, 7:21 PM.

Details

Summary

The i386/ELF JITLink backend was not correctly handling the GOTPC relocation
by skipping the in-built addend, which was manifesting itself in the form of
a segmentation fault in the LF_external_to_absolute_conversion.s test. This
CR has fixed that issue.

Diff Detail

Event Timeline

jain98 created this revision.Jan 13 2023, 7:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2023, 7:21 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
jain98 requested review of this revision.Jan 13 2023, 7:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 13 2023, 7:21 PM
lhames accepted this revision.Jan 13 2023, 7:47 PM

LGTM. Thanks @jain98!

This revision is now accepted and ready to land.Jan 13 2023, 7:47 PM
This revision was landed with ongoing or failed builds.Jan 13 2023, 9:02 PM
This revision was automatically updated to reflect the committed changes.