This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] [test] Mark dfsan tests XFAIL on glibc-2.37
ClosedPublic

Authored by mgorny on Apr 16 2023, 10:59 PM.

Details

Summary

Mark the two dfsan tests that are known to be broken on glibc-2.37
as XFAIL, to make the test suite start passing on Gentoo again.

Bug: https://github.com/llvm/llvm-project/issues/60678

Diff Detail

Event Timeline

mgorny created this revision.Apr 16 2023, 10:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 16 2023, 10:59 PM
Herald added subscribers: Enna1, dberris. · View Herald Transcript
mgorny requested review of this revision.Apr 16 2023, 10:59 PM
thesamesam accepted this revision.Apr 16 2023, 11:02 PM

Yes, please. Obviously we want to fix it properly but right now it's a known issue and it's noise when checking for regressions weekly.

This revision is now accepted and ready to land.Apr 16 2023, 11:02 PM
browneee accepted this revision.Apr 17 2023, 9:23 AM

LGTM

release_shadow_space.c:29: size_t get_rss_kb(): Assertion `feof(f)' failed.

custom.cpp:1858: void test_sprintf(): Assertion `strcmp(buf, "Hello world!") == 0' failed.

Issue appears to be with the program rather than the instrumentation. Did the behavior of the the libc functions used in the test change?

MaskRay accepted this revision.Apr 17 2023, 12:53 PM

Issue appears to be with the program rather than the instrumentation. Did the behavior of the the libc functions used in the test change?

If the question was directed at me, then I'm afraid I don't know.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptApr 17 2023, 6:38 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript