This is an archive of the discontinued LLVM Phabricator instance.

[PPC][PPC64] Improve some llvm-objdump -d -D tests
ClosedPublic

Authored by MaskRay on Apr 23 2019, 1:46 AM.

Details

Summary

Various improvement:

Some offsets in disassembly are incorrect after several layout adjustment. Fix them.
llvm-objdump -D should not be used. -D dumps unrelated non-text sections. Replace them with llvm-objdump -d, llvm-readelf -x, etc
Many llvm-objdump -d tests use {{.*}} . Add the option --no-show-raw-insn to avoid check hex bytes.

ppc64-long-branch.s does not need a shared object. Delete it.
Make ppc64-ifunc.s check 2 ifuncs.

Diff Detail

Repository
rLLD LLVM Linker

Event Timeline

MaskRay created this revision.Apr 23 2019, 1:46 AM
Herald added a project: Restricted Project. · View Herald Transcript
ruiu accepted this revision.Apr 23 2019, 1:58 AM

LGTM

Thank you for doing this.

This revision is now accepted and ready to land.Apr 23 2019, 1:58 AM
MaskRay updated this revision to Diff 196209.Apr 23 2019, 4:44 AM
MaskRay edited the summary of this revision. (Show Details)

Update more tests

This revision was automatically updated to reflect the committed changes.