This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [Patch] p-exp.y: Typecast left operand of BINOP_DIV to long double
- From: Daniel Jacobowitz <drow at false dot org>
- To: Pierre Muller <muller at ics dot u-strasbg dot fr>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 31 Jan 2008 17:26:41 -0500
- Subject: Re: [Patch] p-exp.y: Typecast left operand of BINOP_DIV to long double
- References: <000c01c861c5$149352b0$3db9f810$@u-strasbg.fr>
On Mon, Jan 28, 2008 at 04:47:26PM +0100, Pierre Muller wrote:
> Could someone please confirm that the
> way I inserted the implicit typecast in the parser is
> correct?
> All comments most welcome.
It looks OK to me. Does the type of the left operand completely
determine the type of the operation, or do you need to worry about the
type of the right-hand exp too before you cast?
> @@ -367,7 +370,15 @@ exp : exp '*' exp
> { write_exp_elt_opcode (BINOP_MUL); }
> ;
>
> -exp : exp '/' exp
> +exp : exp '/' {
> + if (current_type && is_integral_type
> (current_type))
> + {
> + write_exp_elt_opcode (UNOP_CAST);
> + write_exp_elt_type (builtin_type_long_double);
> + write_exp_elt_opcode (UNOP_CAST);
> + }
> + }
> + exp
> { write_exp_elt_opcode (BINOP_DIV); }
> ;
>
Won't current_type still be integral after this? Try 1 / 2 / 3.
--
Daniel Jacobowitz
CodeSourcery