This is an archive of the discontinued LLVM Phabricator instance.

[llvm-exegesis] Print errno on failures in subprocess
ClosedPublic

Authored by aidengrossman on Aug 11 2023, 1:01 AM.

Details

Summary

Some error logging in llvm-exegesis under the subprocess executor just
prints a generic failure information rather than any details about the
error as we omit printing the string version of errno. This patch adds
in printing errno at all relevant points in the subprocess executor that
were previously missed.

Diff Detail

Event Timeline

aidengrossman created this revision.Aug 11 2023, 1:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 1:01 AM
Herald added a subscriber: mstojanovic. · View Herald Transcript
aidengrossman requested review of this revision.Aug 11 2023, 1:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 11 2023, 1:01 AM

Missed a couple of these during the switch away from pidfd_getfd and that made debugging a little difficult when I had things setup incorrectly while pulling in Exegesis as a library in an external project.

I didn't add any lit tests here as I don't think there's a way to invoke these failure conditions without relying on buggy behavior.

Bump on this when reviewers have a chance. Thanks!

courbet accepted this revision.Sep 5 2023, 5:17 AM
This revision is now accepted and ready to land.Sep 5 2023, 5:17 AM