This is an archive of the discontinued LLVM Phabricator instance.

[cross-project-tests] Add REQUIRES: compiler-rt to tests that use asan
ClosedPublic

Authored by Orlando on Feb 2 2022, 1:38 AM.

Details

Summary

And XFAIL debuginfo-tests/llgdb-tests/asan-deque.cpp on !system-darwin.
On non-darwin systems these tests use gdb and this one fails because gdb
doesn't pretty-print std::deque (the elements of the deque are not printed so
the CHECK lines fail).

Diff Detail

Event Timeline

Orlando created this revision.Feb 2 2022, 1:38 AM
Orlando requested review of this revision.Feb 2 2022, 1:38 AM

+ llvm-commits

dyung added a subscriber: dyung.Feb 3 2022, 1:14 AM

We've now got a native Linux builder in staging where you can see these tests failing: https://lab.llvm.org/staging/#/builders/205/builds/91

Orlando updated this revision to Diff 407107.Feb 9 2022, 3:06 AM

+ Rebase (fix conflicts with D118468)

LGTM apart from the comment below.

cross-project-tests/debuginfo-tests/llgdb-tests/asan-deque.cpp
7–8

The second expression seems redundant, if !system-darwin && gdb-clang-incompatibility is true then !system-darwin is necessarily also true. Is this meant to be !system-darwin, gdb-clang-incompatibility?

Orlando added inline comments.Feb 9 2022, 4:34 AM
cross-project-tests/debuginfo-tests/llgdb-tests/asan-deque.cpp
7–8

It is redundant yes. I figured it would be more obvious to anyone adjusting the test in the future that the second part is still necessary when the tests start passing on !system-darwin. wdyt?

StephenTozer added inline comments.Feb 9 2022, 8:48 AM
cross-project-tests/debuginfo-tests/llgdb-tests/asan-deque.cpp
7–8

Seems reasonable, but might be worth expanding the below comment to specify that it applies to the !system-darwin clause, since system-darwin being essentially a proxy for "run with LLDB instead of GDB" here might not be obvious to someone unfamiliar with these tests. Something like "failing on non-darwin (gdb) builds because gdb doesn't pretty-print std::deque" or something along those lines.

Orlando updated this revision to Diff 407187.Feb 9 2022, 9:25 AM
Orlando edited the summary of this revision. (Show Details)

+ Improve comment in test
+ Fix description

Orlando marked 2 inline comments as done.Feb 9 2022, 9:25 AM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 10 2022, 2:49 AM
This revision was automatically updated to reflect the committed changes.