Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [PATCH] Enable process record testsuite for RISC-V
@ 2025-10-01 15:10 timurgol007
  2025-10-01 17:15 ` Guinevere Larsen
  0 siblings, 1 reply; 4+ messages in thread
From: timurgol007 @ 2025-10-01 15:10 UTC (permalink / raw)
  To: gdb-patches; +Cc: timurgol007

When I ran GDB testsuite, I noticed that process record tests are not
currently supported on RISC-V. This patch fixes it.
---
 gdb/testsuite/lib/gdb.exp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 94978504a5b..797591801b3 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -3905,7 +3905,8 @@ proc supports_process_record {} {
          || [istarget "aarch64*-*-linux*"]
 	 || [istarget "loongarch*-*-linux*"]
          || [istarget "powerpc*-*-linux*"]
-         || [istarget "s390*-*-linux*"] } {
+	 || [istarget "s390*-*-linux*"]
+	 || [istarget "riscv*-*-*"] } {
 	return 1
     }
 
-- 
2.34.1


^ permalink raw reply	[flat|nested] 4+ messages in thread
* Re: [PATCH] Enable process record testsuite for RISC-V
  2025-10-01 15:10 [PATCH] Enable process record testsuite for RISC-V timurgol007
@ 2025-10-01 17:15 ` Guinevere Larsen
  2025-10-02 23:35   ` timurgol007
  2025-10-02 23:36   ` [pushed][PATCH] " timurgol007
  0 siblings, 2 replies; 4+ messages in thread
From: Guinevere Larsen @ 2025-10-01 17:15 UTC (permalink / raw)
  To: timurgol007, gdb-patches

On 10/1/25 12:10 PM, timurgol007 wrote:
> When I ran GDB testsuite, I noticed that process record tests are not
> currently supported on RISC-V. This patch fixes it.
> ---
>   gdb/testsuite/lib/gdb.exp | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
> index 94978504a5b..797591801b3 100644
> --- a/gdb/testsuite/lib/gdb.exp
> +++ b/gdb/testsuite/lib/gdb.exp
> @@ -3905,7 +3905,8 @@ proc supports_process_record {} {
>            || [istarget "aarch64*-*-linux*"]
>   	 || [istarget "loongarch*-*-linux*"]
>            || [istarget "powerpc*-*-linux*"]
> -         || [istarget "s390*-*-linux*"] } {
> +	 || [istarget "s390*-*-linux*"]
> +	 || [istarget "riscv*-*-*"] } {
>   	return 1
>       }
>   

Hi Timur!

Thanks for noticing this, feel free to push it!

Approved-By: Guinevere Larsen <guinevere@redhat.com>

-- 
Cheers,
Guinevere Larsen
It/she


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [PATCH] Enable process record testsuite for RISC-V
  2025-10-01 17:15 ` Guinevere Larsen
@ 2025-10-02 23:35   ` timurgol007
  2025-10-02 23:36   ` [pushed][PATCH] " timurgol007
  1 sibling, 0 replies; 4+ messages in thread
From: timurgol007 @ 2025-10-02 23:35 UTC (permalink / raw)
  To: gdb-patches, guinevere; +Cc: timurgol007

When I ran GDB testsuite, I noticed that process record tests are not
currently supported on RISC-V. This patch fixes it.

Approved-By: Guinevere Larsen <guinevere@redhat.com>

Hi Guinevere!

Thanks for quick review. Yes, I thought about pushing this patch as
trivial, but decided to play it safe.

---
 gdb/testsuite/lib/gdb.exp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 94978504a5b..797591801b3 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -3905,7 +3905,8 @@ proc supports_process_record {} {
          || [istarget "aarch64*-*-linux*"]
 	 || [istarget "loongarch*-*-linux*"]
          || [istarget "powerpc*-*-linux*"]
-         || [istarget "s390*-*-linux*"] } {
+	 || [istarget "s390*-*-linux*"]
+	 || [istarget "riscv*-*-*"] } {
 	return 1
     }

--
2.34.1


^ permalink raw reply	[flat|nested] 4+ messages in thread
* [pushed][PATCH] Enable process record testsuite for RISC-V
  2025-10-01 17:15 ` Guinevere Larsen
  2025-10-02 23:35   ` timurgol007
@ 2025-10-02 23:36   ` timurgol007
  1 sibling, 0 replies; 4+ messages in thread
From: timurgol007 @ 2025-10-02 23:36 UTC (permalink / raw)
  To: gdb-patches, guinevere; +Cc: timurgol007

When I ran GDB testsuite, I noticed that process record tests are not
currently supported on RISC-V. This patch fixes it.

Approved-By: Guinevere Larsen <guinevere@redhat.com>
---
 gdb/testsuite/lib/gdb.exp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/gdb/testsuite/lib/gdb.exp b/gdb/testsuite/lib/gdb.exp
index 94978504a5b..797591801b3 100644
--- a/gdb/testsuite/lib/gdb.exp
+++ b/gdb/testsuite/lib/gdb.exp
@@ -3905,7 +3905,8 @@ proc supports_process_record {} {
          || [istarget "aarch64*-*-linux*"]
 	 || [istarget "loongarch*-*-linux*"]
          || [istarget "powerpc*-*-linux*"]
-         || [istarget "s390*-*-linux*"] } {
+	 || [istarget "s390*-*-linux*"]
+	 || [istarget "riscv*-*-*"] } {
 	return 1
     }

--
2.34.1


^ permalink raw reply	[flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-10-02 23:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-10-01 15:10 [PATCH] Enable process record testsuite for RISC-V timurgol007
2025-10-01 17:15 ` Guinevere Larsen
2025-10-02 23:35   ` timurgol007
2025-10-02 23:36   ` [pushed][PATCH] " timurgol007

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox