RISC-V: Fix compiler warning of riscv_arg_has_vector

Message ID 20230620094517.3693077-1-lehua.ding@rivai.ai
State Unresolved
Headers
Series RISC-V: Fix compiler warning of riscv_arg_has_vector |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

Lehua Ding June 20, 2023, 9:45 a.m. UTC
  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(-)
  

Comments

juzhe.zhong@rivai.ai June 20, 2023, 9:54 a.m. UTC | #1
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
  
Robin Dapp June 20, 2023, 10:47 a.m. UTC | #2
> 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
  
juzhe.zhong@rivai.ai June 20, 2023, 10:51 a.m. UTC | #3
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
  
Robin Dapp June 20, 2023, 10:56 a.m. UTC | #4
> 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
  
Robin Dapp June 20, 2023, 1:05 p.m. UTC | #5
> Could you merge it ?

Committed.

Regards
 Robin
  
Jeff Law June 20, 2023, 1:11 p.m. UTC | #6
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.
  
Lehua Ding June 20, 2023, 1:22 p.m. UTC | #7
> 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
  
Mark Wielaard June 20, 2023, 2:36 p.m. UTC | #8
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
  
Lehua Ding June 20, 2023, 3:28 p.m. UTC | #9
> All done. Welcome Lehua.
I have received the system notification email, thank you very much.


Best,
Lehua
  

Patch

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;