This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][MC] Corrected branch relocation handling to detect undefined labels
ClosedPublic

Authored by dp on May 14 2020, 7:05 AM.

Details

Summary

Fixed ELF object writer to die gracefully when an undefined label is encountered in a branch instruction.

See bug 41914.

Diff Detail

Event Timeline

dp created this revision.May 14 2020, 7:05 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 14 2020, 7:05 AM
rampitec added inline comments.May 14 2020, 10:45 AM
llvm/lib/Target/AMDGPU/MCTargetDesc/AMDGPUELFObjectWriter.cpp
13

Includes shall be sorted alphabetically.

foad added a subscriber: foad.May 14 2020, 11:03 AM

Typo in summary "udefined".

dp updated this revision to Diff 264047.May 14 2020, 12:03 PM
dp retitled this revision from [AMDGPU][MC] Corrected branch relocation handling to detect udefined labels to [AMDGPU][MC] Corrected branch relocation handling to detect undefined labels.

Corrected a list of included files; fixed a typo in the title.

dp marked an inline comment as done.May 14 2020, 12:09 PM
This revision is now accepted and ready to land.May 14 2020, 1:08 PM
This revision was automatically updated to reflect the committed changes.