[PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Lehua Ding]

Hi,

This little patch fixes a compile warning issue that my previous patch introduced, sorry for introducing this issue.

Best,
Lehua

gcc/ChangeLog:

        * config/riscv/riscv.cc (riscv_arg_has_vector): Add default branch.

---
 gcc/config/riscv/riscv.cc | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc
index 6eb63a9d4de7..9558e28de3fc 100644
--- a/gcc/config/riscv/riscv.cc
+++ b/gcc/config/riscv/riscv.cc
@@ -3820,8 +3820,8 @@ riscv_arg_has_vector (const_tree type)
   switch (TREE_CODE (type))
     {
     case RECORD_TYPE:
-      /* If it is a record, it is further determined whether its fileds have
-         vector type.  */
+      /* If it is a record, it is further determined whether its fields have
+	 vector type.  */
       for (tree f = TYPE_FIELDS (type); f; f = DECL_CHAIN (f))
 	if (TREE_CODE (f) == FIELD_DECL)
 	  {
@@ -3835,6 +3835,8 @@ riscv_arg_has_vector (const_tree type)
       break;
     case ARRAY_TYPE:
       return riscv_arg_has_vector (TREE_TYPE (type));
+    default:
+      break;
     }
 
   return false;
-- 
2.36.3

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[juzhe.zhong]

[-- Attachment #1: Type: text/plain, Size: 1413 bytes --]

Ok.



juzhe.zhong@rivai.ai
 
From: Lehua Ding
Date: 2023-06-20 17:45
To: gcc-patches
CC: juzhe.zhong; rdapp.gcc; kito.cheng; palmer; jeffreyalaw; pan2.li
Subject: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector
Hi,
 
This little patch fixes a compile warning issue that my previous patch introduced, sorry for introducing this issue.
 
Best,
Lehua
 
gcc/ChangeLog:
 
        * config/riscv/riscv.cc (riscv_arg_has_vector): Add default branch.
 
---
gcc/config/riscv/riscv.cc | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
 
diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc
index 6eb63a9d4de7..9558e28de3fc 100644
--- a/gcc/config/riscv/riscv.cc
+++ b/gcc/config/riscv/riscv.cc
@@ -3820,8 +3820,8 @@ riscv_arg_has_vector (const_tree type)
   switch (TREE_CODE (type))
     {
     case RECORD_TYPE:
-      /* If it is a record, it is further determined whether its fileds have
-         vector type.  */
+      /* If it is a record, it is further determined whether its fields have
+ vector type.  */
       for (tree f = TYPE_FIELDS (type); f; f = DECL_CHAIN (f))
if (TREE_CODE (f) == FIELD_DECL)
  {
@@ -3835,6 +3835,8 @@ riscv_arg_has_vector (const_tree type)
       break;
     case ARRAY_TYPE:
       return riscv_arg_has_vector (TREE_TYPE (type));
+    default:
+      break;
     }
   return false;
-- 
2.36.3
 

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Robin Dapp]

> This little patch fixes a compile warning issue that my previous
> patch introduced, sorry for introducing this issue.

OK and obvious enough to push directly.

Regards
 Robin

Re: Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[juzhe.zhong]

[-- Attachment #1: Type: text/plain, Size: 514 bytes --]

Could you merge it ?
By the way, could Lehua get the write access?

Thanks.


juzhe.zhong@rivai.ai
 
From: Robin Dapp
Date: 2023-06-20 18:47
To: Lehua Ding; gcc-patches
CC: rdapp.gcc; juzhe.zhong; kito.cheng; palmer; jeffreyalaw; pan2.li
Subject: Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector
> This little patch fixes a compile warning issue that my previous
> patch introduced, sorry for introducing this issue.
 
OK and obvious enough to push directly.
 
Regards
Robin
 

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Robin Dapp]

> Could you merge it ?
> By the way, could Lehua get the write access?

IMHO nothing stands in the way but I'll defer to Jeff to have
the "official seal" :)
Once he ACKs Lehua needs to go the usual way of requesting
sourceware access via https://sourceware.org/cgi-bin/pdw/ps_form.cgi.

Regards
 Robin

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Robin Dapp]

> Could you merge it ?

Committed.

Regards
 Robin

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Jeff Law]

On 6/20/23 04:56, Robin Dapp wrote:
>> Could you merge it ?
>> By the way, could Lehua get the write access?
> 
> IMHO nothing stands in the way but I'll defer to Jeff to have
> the "official seal" :)
> Once he ACKs Lehua needs to go the usual way of requesting
> sourceware access via https://sourceware.org/cgi-bin/pdw/ps_form.cgi.
Lehua fills out that form.  List me as the approver and the process will 
run from there.  Takes a day or two for everything to get into place.

jeff

ps.  If Lehua has already filled out the form with Robin as the 
approver, that's fine too.  Might take a bit longer as I suspect the IT 
folks may not recognize Robin.

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Lehua Ding]

[-- Attachment #1: Type: text/plain, Size: 265 bytes --]

> Lehua fills out that form.  List me as the approver and the process will
> run from there.  Takes a day or two for everything to get into place.


I just followed this step to submit the form, thanks to Robin, Jeff and Juzhe.


Best,
Lehua

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[Mark Wielaard]

Hi all,

On Tue, 2023-06-20 at 07:11 -0600, Jeff Law wrote:
> On 6/20/23 04:56, Robin Dapp wrote:
> > > Could you merge it ?
> > > By the way, could Lehua get the write access?
> > 
> > IMHO nothing stands in the way but I'll defer to Jeff to have
> > the "official seal" :)
> > Once he ACKs Lehua needs to go the usual way of requesting
> > sourceware access via https://sourceware.org/cgi-bin/pdw/ps_form.cgi.
> Lehua fills out that form.  List me as the approver and the process will 
> run from there.  Takes a day or two for everything to get into place.

All done. Welcome Lehua.

> ps.  If Lehua has already filled out the form with Robin as the > approver, that's fine too.  Might take a bit longer as I suspect the
> IT folks may not recognize Robin. 

Also Robin is right, you are on the hook as approver for the "official
seal" :) Because the "IT folks" check that the approver is listed as a
gcc maintainer and not just has write after approval status.

Cheers,

Mark

Re: [PATCH] RISC-V: Fix compiler warning of riscv_arg_has_vector

[=?gb18030?B?TGVodWEgRGluZw==?=]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb18030", Size: 121 bytes --]

> All done. Welcome Lehua.
I have received the system notification email, thank you very much.


Best,
Lehua