* [PATCH] RISC-V: Remove support for non-existing 'Zve32d'
@ 2023-08-03 1:19 Tsukasa OI
2023-08-09 23:37 ` Tsukasa OI
2023-08-11 0:38 ` Nelson Chu
0 siblings, 2 replies; 5+ messages in thread
From: Tsukasa OI @ 2023-08-03 1:19 UTC (permalink / raw)
To: Tsukasa OI, Palmer Dabbelt, Andrew Waterman, Jim Wilson,
Nelson Chu, Kito Cheng
Cc: binutils
From: Tsukasa OI <research_trasio@irq.a4lg.com>
Since this "extension" does not exist (on the other hand, 'Zve64d' exists)
and it's not useful if we keep it (as other code portions just ignore
"zve32d"), this commit just removes it.
bfd/ChangeLog:
* elfxx-riscv.c (riscv_supported_std_z_ext): Remove 'Zve32d'
extension from the list.
---
bfd/elfxx-riscv.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
index ba5165766b2b..1971e6ba6a6b 100644
--- a/bfd/elfxx-riscv.c
+++ b/bfd/elfxx-riscv.c
@@ -1280,7 +1280,6 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] =
{"zkt", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
{"zve32x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
{"zve32f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
- {"zve32d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
{"zve64x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
{"zve64f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
{"zve64d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
base-commit: 4b177a76d5b759ba631568fb69e8750e99b43647
--
2.41.0
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RISC-V: Remove support for non-existing 'Zve32d'
2023-08-03 1:19 [PATCH] RISC-V: Remove support for non-existing 'Zve32d' Tsukasa OI
@ 2023-08-09 23:37 ` Tsukasa OI
2023-08-11 0:38 ` Nelson Chu
1 sibling, 0 replies; 5+ messages in thread
From: Tsukasa OI @ 2023-08-09 23:37 UTC (permalink / raw)
To: Palmer Dabbelt, Andrew Waterman, Jim Wilson, Nelson Chu, Kito Cheng
Cc: binutils
Note that GCC counterpart is merged.
<https://gcc.gnu.org/pipermail/gcc-patches/2023-August/626747.html>
<https://gcc.gnu.org/pipermail/gcc-patches/2023-August/626862.html>
Thanks,
Tsukasa
On 2023/08/03 10:19, Tsukasa OI wrote:
> From: Tsukasa OI <research_trasio@irq.a4lg.com>
>
> Since this "extension" does not exist (on the other hand, 'Zve64d' exists)
> and it's not useful if we keep it (as other code portions just ignore
> "zve32d"), this commit just removes it.
>
> bfd/ChangeLog:
>
> * elfxx-riscv.c (riscv_supported_std_z_ext): Remove 'Zve32d'
> extension from the list.
> ---
> bfd/elfxx-riscv.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index ba5165766b2b..1971e6ba6a6b 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1280,7 +1280,6 @@ static struct riscv_supported_ext riscv_supported_std_z_ext[] =
> {"zkt", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> - {"zve32d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>
> base-commit: 4b177a76d5b759ba631568fb69e8750e99b43647
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RISC-V: Remove support for non-existing 'Zve32d'
2023-08-03 1:19 [PATCH] RISC-V: Remove support for non-existing 'Zve32d' Tsukasa OI
2023-08-09 23:37 ` Tsukasa OI
@ 2023-08-11 0:38 ` Nelson Chu
2023-08-11 1:51 ` Palmer Dabbelt
2023-08-11 2:00 ` Tsukasa OI
1 sibling, 2 replies; 5+ messages in thread
From: Nelson Chu @ 2023-08-11 0:38 UTC (permalink / raw)
To: Tsukasa OI
Cc: Palmer Dabbelt, Andrew Waterman, Jim Wilson, Kito Cheng, binutils
[-- Attachment #1: Type: text/plain, Size: 1501 bytes --]
Okay, seems it's just a typo,
https://github.com/riscv/riscv-v-spec/commit/3af89ab16dd0ea85b15db35a2cd3c94aae5859a5
.
Thanks
Nelson
On Thu, Aug 3, 2023 at 9:19 AM Tsukasa OI <research_trasio@irq.a4lg.com>
wrote:
> From: Tsukasa OI <research_trasio@irq.a4lg.com>
>
> Since this "extension" does not exist (on the other hand, 'Zve64d' exists)
> and it's not useful if we keep it (as other code portions just ignore
> "zve32d"), this commit just removes it.
>
> bfd/ChangeLog:
>
> * elfxx-riscv.c (riscv_supported_std_z_ext): Remove 'Zve32d'
> extension from the list.
> ---
> bfd/elfxx-riscv.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index ba5165766b2b..1971e6ba6a6b 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1280,7 +1280,6 @@ static struct riscv_supported_ext
> riscv_supported_std_z_ext[] =
> {"zkt", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> - {"zve32d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>
> base-commit: 4b177a76d5b759ba631568fb69e8750e99b43647
> --
> 2.41.0
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RISC-V: Remove support for non-existing 'Zve32d'
2023-08-11 0:38 ` Nelson Chu
@ 2023-08-11 1:51 ` Palmer Dabbelt
2023-08-11 2:00 ` Tsukasa OI
1 sibling, 0 replies; 5+ messages in thread
From: Palmer Dabbelt @ 2023-08-11 1:51 UTC (permalink / raw)
To: nelson; +Cc: research_trasio, Andrew Waterman, Jim Wilson, kito.cheng, binutils
On Thu, 10 Aug 2023 17:38:37 PDT (-0700), nelson@rivosinc.com wrote:
> Okay, seems it's just a typo,
> https://github.com/riscv/riscv-v-spec/commit/3af89ab16dd0ea85b15db35a2cd3c94aae5859a5
> .
Ya
Reviewed-by: Palmer Dabbelt <palmer@rivosinc.com>
>
> Thanks
> Nelson
>
> On Thu, Aug 3, 2023 at 9:19 AM Tsukasa OI <research_trasio@irq.a4lg.com>
> wrote:
>
>> From: Tsukasa OI <research_trasio@irq.a4lg.com>
>>
>> Since this "extension" does not exist (on the other hand, 'Zve64d' exists)
>> and it's not useful if we keep it (as other code portions just ignore
>> "zve32d"), this commit just removes it.
>>
>> bfd/ChangeLog:
>>
>> * elfxx-riscv.c (riscv_supported_std_z_ext): Remove 'Zve32d'
>> extension from the list.
>> ---
>> bfd/elfxx-riscv.c | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
>> index ba5165766b2b..1971e6ba6a6b 100644
>> --- a/bfd/elfxx-riscv.c
>> +++ b/bfd/elfxx-riscv.c
>> @@ -1280,7 +1280,6 @@ static struct riscv_supported_ext
>> riscv_supported_std_z_ext[] =
>> {"zkt", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> {"zve32x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> {"zve32f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> - {"zve32d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> {"zve64x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> {"zve64f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>> {"zve64d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>>
>> base-commit: 4b177a76d5b759ba631568fb69e8750e99b43647
>> --
>> 2.41.0
>>
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] RISC-V: Remove support for non-existing 'Zve32d'
2023-08-11 0:38 ` Nelson Chu
2023-08-11 1:51 ` Palmer Dabbelt
@ 2023-08-11 2:00 ` Tsukasa OI
1 sibling, 0 replies; 5+ messages in thread
From: Tsukasa OI @ 2023-08-11 2:00 UTC (permalink / raw)
To: Nelson Chu, Palmer Dabbelt; +Cc: binutils
On 2023/08/11 9:38, Nelson Chu wrote:
> Okay, seems it's just a
> typo, https://github.com/riscv/riscv-v-spec/commit/3af89ab16dd0ea85b15db35a2cd3c94aae5859a5 <https://github.com/riscv/riscv-v-spec/commit/3af89ab16dd0ea85b15db35a2cd3c94aae5859a5>.
Oh, that was also a very "close-to-ratified-1.0" change.
On 2023/08/11 10:51, Palmer Dabbelt wrote:
> Reviewed-by: Palmer Dabbelt <palmer@rivosinc.com>
Thanks you two for reviewing! Committed.
Thanks,
Tsukasa
>
> Thanks
> Nelson
>
> On Thu, Aug 3, 2023 at 9:19 AM Tsukasa OI <research_trasio@irq.a4lg.com
> <mailto:research_trasio@irq.a4lg.com>> wrote:
>
> From: Tsukasa OI <research_trasio@irq.a4lg.com
> <mailto:research_trasio@irq.a4lg.com>>
>
> Since this "extension" does not exist (on the other hand, 'Zve64d'
> exists)
> and it's not useful if we keep it (as other code portions just ignore
> "zve32d"), this commit just removes it.
>
> bfd/ChangeLog:
>
> * elfxx-riscv.c (riscv_supported_std_z_ext): Remove 'Zve32d'
> extension from the list.
> ---
> bfd/elfxx-riscv.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/bfd/elfxx-riscv.c b/bfd/elfxx-riscv.c
> index ba5165766b2b..1971e6ba6a6b 100644
> --- a/bfd/elfxx-riscv.c
> +++ b/bfd/elfxx-riscv.c
> @@ -1280,7 +1280,6 @@ static struct riscv_supported_ext
> riscv_supported_std_z_ext[] =
> {"zkt", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve32f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> - {"zve32d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64x", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64f", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
> {"zve64d", ISA_SPEC_CLASS_DRAFT, 1, 0, 0 },
>
> base-commit: 4b177a76d5b759ba631568fb69e8750e99b43647
> --
> 2.41.0
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-08-11 2:00 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-08-03 1:19 [PATCH] RISC-V: Remove support for non-existing 'Zve32d' Tsukasa OI
2023-08-09 23:37 ` Tsukasa OI
2023-08-11 0:38 ` Nelson Chu
2023-08-11 1:51 ` Palmer Dabbelt
2023-08-11 2:00 ` Tsukasa OI
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).