RE: [PATCH v3 00/11] Improve Fortran intrinsic types and procedures

["Kempke, Nils-Christian" <nils-christian.kempke@intel.com>]

> From: Gdb-patches <gdb-patches-bounces+nils-
> christian.kempke=intel.com@sourceware.org> On Behalf Of Kempke, Nils-
> Christian via Gdb-patches
> Sent: Monday, April 25, 2022 3:20 PM
> To: Luis Machado <luis.machado@arm.com>; gdb-patches@sourceware.org
> Subject: RE: [PATCH v3 00/11] Improve Fortran intrinsic types and procedures
> 
> Hi,
> 
> Sorry for that - I think the assert should have been removed anyway.
> 
> I might have missed removing it - but the two lines below are what I wrote
> instead - an
> error and gdb should not crash..
> 
>   if (value_type (arg1)->code () == TYPE_CODE_COMPLEX
>       || value_type (arg2)->code () == TYPE_CODE_COMPLEX)
>     error (_("Types of arguments for CMPLX called with more then one
> argument "
> 	     "must be REAL or INTEGER"));
> 
> Does removing the assert do that? I'd assume the tests then just FAIL.
> 
> Cheers,
> Nils

I have to correct myself - I wanted to say we should instead of having the assert
likely have an error instead, similar to the two lines below (the ones I cited).

Nils

Intel Deutschland GmbH
Registered Address: Am Campeon 10, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de <http://www.intel.de>
Managing Directors: Christin Eisenschmid, Sharon Heck, Tiffany Doon Silva  
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928