This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Patch to (almost) completly update frv port to use target hooks
- From: "Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>
- To: graham dot stott at btinternet dot com
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 5 Sep 2002 01:14:58 -0400 (EDT)
- Subject: Re: Patch to (almost) completly update frv port to use target hooks
- References: <200209041911.PAA07283@caip.rutgers.edu> <3D767819.5070008@btinternet.com>
> From: Graham Stott <graham.stott@btinternet.com>
>
> So frv_encode_section_info was only called if FIRST was TRUE
> but your change to frv_encode_section_info means that if first
> is TRUE then the function body is not executed.
Ug, I think you're right. Thanks for catching my mistake.
I checked in the obvious fix.
Index: ChangeLog
===================================================================
RCS file: /cvs/gcc/gcc/gcc/ChangeLog,v
retrieving revision 1.15317
diff -u -p -r1.15317 ChangeLog
--- ChangeLog 5 Sep 2002 03:49:38 -0000 1.15317
+++ ChangeLog 5 Sep 2002 05:10:39 -0000
@@ -1,3 +1,7 @@
+2002-09-04 Kaveh R. Ghazi <ghazi@caip.rutgers.edu>
+
+ * frv.c (frv_encode_section_info): Fix error in last change.
+
2002-09-04 David Edelsohn <edelsohn@gnu.org>
* config/rs6000/rs6000.c (rs6000_flag_pic): New variable.
Index: config/frv/frv.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/config/frv/frv.c,v
retrieving revision 1.3
diff -u -p -r1.3 frv.c
--- config/frv/frv.c 4 Sep 2002 20:56:33 -0000 1.3
+++ config/frv/frv.c 5 Sep 2002 05:10:41 -0000
@@ -741,7 +741,7 @@ frv_encode_section_info (decl, first)
tree decl;
int first;
{
- if (first)
+ if (! first)
return;
if (TREE_CODE (decl) == VAR_DECL)
{