This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Re: Some type mismatches


On Thu, 31 May 2007, FX Coudert wrote:

> >-  add_sym_1 ("sizeof", GFC_ISYM_SIZEOF, NOT_ELEMENTAL, ACTUAL_NO,
> >BT_INTEGER, di,
> >+  add_sym_1 ("sizeof", GFC_ISYM_SIZEOF, NOT_ELEMENTAL, ACTUAL_NO,
> >BT_INTEGER, ii,
> >	     GFC_STD_GNU, gfc_check_sizeof, NULL, NULL,
> >-	     i, BT_INTEGER, di, REQUIRED);
> >+	     i, BT_INTEGER, ii, REQUIRED);
> >the argument types are ignored (which is the purpose of the empty check
> >function).
> 
> I was about to send a mail to OK it also, but we should really have:
> 
> >-	     i, BT_INTEGER, di, REQUIRED);
> >+	     i, BT_UNKNOWN, 0, REQUIRED);
> 
> instead, so that we see easily that arg types isn't important. This is what is
> done for the LOC intrinsic already, for example.

Fair enough, that works as well.

Richard.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]