* [PATCH] RX removed control register cpen
@ 2020-03-12 17:49 Darius Galis
2020-03-12 19:43 ` Jeff Law
0 siblings, 1 reply; 2+ messages in thread
From: Darius Galis @ 2020-03-12 17:49 UTC (permalink / raw)
To: gcc-patches
Hello,
The following patch removes the CTRLREG_CPEN register variable.
According to the RX software manual: https://www.renesas.com/cn/en/doc/products/mpumcu/doc/rx_family/r01us0316ej0100-rxv3sm.pdf
there is no control register called cpen.
Regression test is OK, without additional fails, and was tested with the
following command:
make -k check-gcc RUNTESTFLAGS=--target_board=rx-sim
Please let me know if this is OK, Thank you!
Darius
From ce78a5cde96d2ace6e06733320edbfb0c59c0e88 Mon Sep 17 00:00:00 2001
From: Darius <darius.galis@cyberthorstudios.com>
Date: Tue, 10 Mar 2020 18:21:20 +0200
Subject: Removed CTRLREG_CPEN register
---
gcc/ChangeLog | 5 +++++
gcc/config/rx/rx.c | 1 -
gcc/config/rx/rx.md | 1 -
3 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 6c4a505..ea9bbb2 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,8 @@
+2020-03-10 Darius Galis <darius.galis@cyberthorstudios.com>
+
+ * config/rx/rx.md: Remove CTRLREG_CPEN
+ * config/rx/rx.c: Likewise.
+
2020-03-09 Martin Liska <mliska@suse.cz>
PR target/93800
diff --git a/gcc/config/rx/rx.c b/gcc/config/rx/rx.c
index 0a51124..151ad39 100644
--- a/gcc/config/rx/rx.c
+++ b/gcc/config/rx/rx.c
@@ -641,7 +641,6 @@ rx_print_operand (FILE * file, rtx op, int letter)
case CTRLREG_PSW: fprintf (file, "psw"); break;
case CTRLREG_USP: fprintf (file, "usp"); break;
case CTRLREG_FPSW: fprintf (file, "fpsw"); break;
- case CTRLREG_CPEN: fprintf (file, "cpen"); break;
case CTRLREG_BPSW: fprintf (file, "bpsw"); break;
case CTRLREG_BPC: fprintf (file, "bpc"); break;
case CTRLREG_ISP: fprintf (file, "isp"); break;
diff --git a/gcc/config/rx/rx.md b/gcc/config/rx/rx.md
index 0cf5025..df08a9e 100644
--- a/gcc/config/rx/rx.md
+++ b/gcc/config/rx/rx.md
@@ -79,7 +79,6 @@
(CTRLREG_PSW 0)
(CTRLREG_USP 2)
(CTRLREG_FPSW 3)
- (CTRLREG_CPEN 4)
(CTRLREG_BPSW 8)
(CTRLREG_BPC 9)
(CTRLREG_ISP 10)
--
1.9.1
--
Ing. Darius Galiș
Software Engineer - CyberTHOR Studios Ltd.
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] RX removed control register cpen
2020-03-12 17:49 [PATCH] RX removed control register cpen Darius Galis
@ 2020-03-12 19:43 ` Jeff Law
0 siblings, 0 replies; 2+ messages in thread
From: Jeff Law @ 2020-03-12 19:43 UTC (permalink / raw)
To: Darius Galis, gcc-patches
On Thu, 2020-03-12 at 19:49 +0200, Darius Galis wrote:
> Hello,
>
> The following patch removes the CTRLREG_CPEN register variable.
> According to the RX software manual:
> https://www.renesas.com/cn/en/doc/products/mpumcu/doc/rx_family/r01us0316ej0100-rxv3sm.pdf
> there is no control register called cpen.
>
> Regression test is OK, without additional fails, and was tested with the
> following command:
>
> make -k check-gcc RUNTESTFLAGS=--target_board=rx-sim
>
> Please let me know if this is OK, Thank you!
> Darius
>
> From ce78a5cde96d2ace6e06733320edbfb0c59c0e88 Mon Sep 17 00:00:00 2001
> From: Darius <darius.galis@cyberthorstudios.com>
> Date: Tue, 10 Mar 2020 18:21:20 +0200
> Subject: Removed CTRLREG_CPEN register
>
> ---
> gcc/ChangeLog | 5 +++++
> gcc/config/rx/rx.c | 1 -
> gcc/config/rx/rx.md | 1 -
> 3 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> index 6c4a505..ea9bbb2 100644
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,8 @@
> +2020-03-10 Darius Galis <darius.galis@cyberthorstudios.com>
> +
> + * config/rx/rx.md: Remove CTRLREG_CPEN
> + * config/rx/rx.c: Likewise.
THanks. I verified that the CPEN register was removed in 2009. Installed on the
trunk.
jeff
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-03-12 19:44 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-12 17:49 [PATCH] RX removed control register cpen Darius Galis
2020-03-12 19:43 ` Jeff Law
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).