This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] initial OpenCL C language support
- From: Tom Tromey <tromey at redhat dot com>
- To: Ken Werner <ken at linux dot vnet dot ibm dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Mon, 25 Oct 2010 16:45:58 -0600
- Subject: Re: [patch] initial OpenCL C language support
- References: <201010221920.30046.ken@linux.vnet.ibm.com>
>>>>> "Ken" == Ken Werner <ken@linux.vnet.ibm.com> writes:
Ken> This patch implements initial GDB support for the OpenCL C
Ken> Programming Language. Since OpenCL is based on C99 I tried to reuse
Ken> GDBs C infrastructure where possible (expression parser, printing
Ken> routines, etc).
I have been skimming this patch a bit and I think it is very good.
I don't have time to fully review it right now, as I'm at the GCC
Summit. But I did notice one thing:
Ken> +/* Expression evaluator for the OpenCL. Most operations are delegated to
Ken> + evaluate_subexp_standard; see that function for a description of the
Ken> + arguments. */
Ken> +static struct value *
Ken> +evaluate_subexp_opencl (struct type *expect_type, struct expression *exp,
Ken> + int *pos, enum noside noside)
[...]
Ken> + return evaluate_subexp_standard (expect_type, exp, pos, noside);
I think this ought to call the C-specific evaluate_subexp_c.
You'll need to at least make this non-static, maybe even rename it.
I believe that without this some forms of string constants will not work
properly.
Tom