Page MenuHomePhabricator

Fix the roundtrip test under llvm-profdata
ClosedPublic

Authored by lxfind on Jun 4 2020, 4:26 PM.

Details

Summary

According to the comments, we want to convert the profile into two binary formats, and then into the md5text format.
We seems to have ignored the intermediate files.
This patch uses them to complete the full roundtrips.

Diff Detail

Event Timeline

lxfind created this revision.Jun 4 2020, 4:26 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 4 2020, 4:26 PM
wmi accepted this revision.Jun 4 2020, 5:11 PM

Thanks for catching the mistake!

This revision is now accepted and ready to land.Jun 4 2020, 5:11 PM
wenlei added a comment.Jun 5 2020, 8:56 AM

@lxfind thanks for the fix, I just committed the patch on your behalf.

This revision was automatically updated to reflect the committed changes.