This is an archive of the discontinued LLVM Phabricator instance.

[DWARFYAML][test] Simplify __debug_pubnames/types tests. NFC.
ClosedPublic

Authored by Higuoxing on Sep 22 2020, 12:18 AM.

Details

Summary

This patch stripped unneeded sections from the test case.

Diff Detail

Event Timeline

Higuoxing created this revision.Sep 22 2020, 12:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2020, 12:18 AM
Higuoxing requested review of this revision.Sep 22 2020, 12:18 AM
grimar accepted this revision.Sep 22 2020, 1:06 AM

I am not an expert in MachO, but this change looks good and reasonable to me. Worth waiting for a second opinion.

This revision is now accepted and ready to land.Sep 22 2020, 1:06 AM
MaskRay accepted this revision.Sep 22 2020, 9:38 AM

LGTM.