gdb/testsuite: add a clang XFAIL to gdb.python/py-watchpoint.exp
authorGuinevere Larsen <blarsen@redhat.com>
Wed, 25 Oct 2023 08:44:19 +0000 (10:44 +0200)
committerGuinevere Larsen <blarsen@redhat.com>
Wed, 25 Oct 2023 16:24:13 +0000 (18:24 +0200)
Clang doesn't use CFA information for variable locations. This makes it
so software breakpoints get a false hit when rbp gets popped, causing
a FAIL in gdb.python/py-watchpoint.exp. Since this is nothing wrong with
GDB itself, add an xfail to reduce noise.

Approved-By: Tom Tromey <tom@tromey.com>
gdb/testsuite/gdb.python/py-watchpoint.exp

index 9a6ef4475722ad44a3e438c1f41ca6e59f83eb13..5ff61285979e5eeb3d230d0e5bf5ed735ceeed2a 100644 (file)
@@ -42,5 +42,20 @@ gdb_test "source $pyfile" ".*Python script imported.*" \
     "import python scripts"
 gdb_test "python print(len(gdb.breakpoints()))" "2" "check modified BP count"
 gdb_test "continue" ".*" "run until program stops"
-gdb_test "python print(bpt.n)" "5" "check watchpoint hits"
+# Clang doesn't use CFA location information for variables (despite generating
+# them), meaning when the instruction "pop rbp" happens, we get a false hit
+# on the watchpoint. for more details, see:
+# https://github.com/llvm/llvm-project/issues/64390
+gdb_test_multiple "python print(bpt.n)" "check watchpoint hits" {
+    -re -wrap "5" {
+       pass $gdb_test_name
+    }
+    -re -wrap "6" {
+       if {[test_compiler_info "clang-*"]} {
+           xfail "$gdb_test_name (clang issue 64390)"
+       } else {
+           fail $gdb_test_name
+       }
+    }
+}
 gdb_test "python print(len(gdb.breakpoints()))" "1" "check BP count"