[llvm] [Dexter] Extend XFAIL of Dexter tests to all MacOS architectures. (PR #83936)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 4 17:31:44 PST 2024
https://github.com/dyung created https://github.com/llvm/llvm-project/pull/83936
I am trying to bring up a MacOS buildbot targeting x86 and noticed that two Dexter tests were failing, `cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp` and `cross-project-tests/debuginfo-tests/llgdb-tests/static-member-2.cpp`. Looking in the history for these tests, they were XFAILed for Apple Silicon in 9c46606. To test, I locally removed the XFAIL on my local Apple Silicon machine, and the errors produced match those my buildbot on x86 is producing.
For example:
```
FAIL: cross-project-tests :: debuginfo-tests/llgdb-tests/static-member.cpp (132 of 137)
******************** TEST 'cross-project-tests :: debuginfo-tests/llgdb-tests/static-member.cpp' FAILED ********************
Exit Code: 1
Command Output (stdout):
--
Debugger output was:
error: 'delete' is not a valid command.
imported lldb from: "/Applications/Xcode.app/Contents/SharedFrameworks/LLDB.framework/Resources/Python"
> delete breakpoints
None
> break static-member.cpp:33
SBBreakpoint: id = 1, file = 'static-member.cpp', line = 33, exact_match = 0, locations = 1
> r
success
> ptype MyClass
class MyClass {
static const int a = 4;
static int b;
static int c;
public:
int d;
}
> p MyClass::a
(const int) $0 = 4
> p MyClass::c
(int) $1 = 15
> quit
--
Command Output (stderr):
--
RUN: at line 1: /Users/dyung/src/git/llvm/build/bin/clang --driver-mode=g++ --target=arm64-apple-darwin23.3.0 -O0 -g /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp -o /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp -c
+ /Users/dyung/src/git/llvm/build/bin/clang --driver-mode=g++ --target=arm64-apple-darwin23.3.0 -O0 -g /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp -o /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp -c
RUN: at line 2: /Users/dyung/src/git/llvm/build/bin/clang --driver-mode=g++ --target=arm64-apple-darwin23.3.0 /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp -o /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp.out
+ /Users/dyung/src/git/llvm/build/bin/clang --driver-mode=g++ --target=arm64-apple-darwin23.3.0 /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp -o /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp.out
RUN: at line 3: /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/test_debuginfo.pl /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp.out
+ /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/test_debuginfo.pl /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.tmp.out
/Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp:11:11: error: CHECK: expected string not found in input
// CHECK: static const int a;
^
/Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.gdb.output:10:16: note: scanning from here
class MyClass {
^
/Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.gdb.output:11:2: note: possible intended match here
static const int a = 4;
^
Input file: /Users/dyung/src/git/llvm/build/projects/cross-project-tests/debuginfo-tests/llgdb-tests/Output/static-member.cpp.gdb.output
Check file: /Users/dyung/src/git/llvm/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp
-dump-input=help explains the following input dump.
Input was:
<<<<<<
.
.
.
5: > break static-member.cpp:33
6: SBBreakpoint: id = 1, file = 'static-member.cpp', line = 33, exact_match = 0, locations = 1
7: > r
8: success
9: > ptype MyClass
10: class MyClass {
check:11'0 X error: no match found
11: static const int a = 4;
check:11'0 ~~~~~~~~~~~~~~~~~~~~~~~~~
check:11'1 ? possible intended match
12: static int b;
check:11'0 ~~~~~~~~~~~~~~~
13: static int c;
check:11'0 ~~~~~~~~~~~~~~~
14: public:
check:11'0 ~~~~~~~~
15: int d;
check:11'0 ~~~~~~~~
16: }
check:11'0 ~~
.
.
.
>>>>>>
```
This matches what my x86 MacOS buildbot seems to be hitting for this test https://lab.llvm.org/staging/#/builders/188/builds/307/steps/6/logs/FAIL__cross-project-tests__static-member_cpp.
The failing output for static-member-2.cpp also seems to match between my Apple Silicon and x86 machines, so I think it makes sense to extend the XFAIL to all `system-darwin` targets, and not just aarch64.
>From 1155ccbce6faceead0e333abb5154a08eea4d6c9 Mon Sep 17 00:00:00 2001
From: Douglas Yung <dyung+github at fastmail.fm>
Date: Mon, 4 Mar 2024 17:25:20 -0800
Subject: [PATCH] [Dexter] Extend XFAIL of Dexter tests to all MacOS
architectures.
---
.../debuginfo-tests/llgdb-tests/static-member-2.cpp | 2 +-
.../debuginfo-tests/llgdb-tests/static-member.cpp | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/cross-project-tests/debuginfo-tests/llgdb-tests/static-member-2.cpp b/cross-project-tests/debuginfo-tests/llgdb-tests/static-member-2.cpp
index 79ff74cb2d0aeb..3f11ae018fc858 100644
--- a/cross-project-tests/debuginfo-tests/llgdb-tests/static-member-2.cpp
+++ b/cross-project-tests/debuginfo-tests/llgdb-tests/static-member-2.cpp
@@ -2,7 +2,7 @@
// RUN: %clangxx %target_itanium_abi_host_triple %t -o %t.out
// RUN: %test_debuginfo %s %t.out
// XFAIL: gdb-clang-incompatibility
-// XFAIL: system-darwin && target-aarch64
+// XFAIL: system-darwin
// DEBUGGER: delete breakpoints
// DEBUGGER: break static-member.cpp:33
diff --git a/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp b/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp
index abfa8e3337f64d..57316dfd640404 100644
--- a/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp
+++ b/cross-project-tests/debuginfo-tests/llgdb-tests/static-member.cpp
@@ -2,7 +2,7 @@
// RUN: %clangxx %target_itanium_abi_host_triple %t -o %t.out
// RUN: %test_debuginfo %s %t.out
// XFAIL: !system-darwin && gdb-clang-incompatibility
-// XFAIL: system-darwin && target-aarch64
+// XFAIL: system-darwin
// DEBUGGER: delete breakpoints
// DEBUGGER: break static-member.cpp:33
// DEBUGGER: r
More information about the llvm-commits
mailing list