This is an archive of the discontinued LLVM Phabricator instance.

[MSAN] Add result printing for failed call in pthread_getaffinity_np. Will be reverted when test failure is diagnosed. Depends on: https://reviews.llvm.org/D127185
ClosedPublic

Authored by kda on Jun 8 2022, 9:47 AM.

Diff Detail

Event Timeline

kda created this revision.Jun 8 2022, 9:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 8 2022, 9:47 AM
Herald added a subscriber: Enna1. · View Herald Transcript
kda requested review of this revision.Jun 8 2022, 9:47 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 8 2022, 9:47 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka accepted this revision.Jun 8 2022, 10:51 AM
vitalybuka added inline comments.
compiler-rt/test/sanitizer_common/TestCases/Linux/pthread_getaffinity_np.cpp
3

unless you already know the reason, maybe enable android as well?

This revision is now accepted and ready to land.Jun 8 2022, 10:51 AM
kda added inline comments.Jun 8 2022, 11:13 AM
compiler-rt/test/sanitizer_common/TestCases/Linux/pthread_getaffinity_np.cpp
3

Yes. This test broke android build, as libc (where android keeps libresolv) does not implement pthread_getaffinity_np. (In fact, the build indicated that it might think I wanted to use 'sched_getaffinity', which is already intercepted.)