[cfe-commits] [PATCH] Add frontend support for OpenCL build options
lattner at apple.com
Mon Nov 29 14:40:25 PST 2010
On Nov 29, 2010, at 2:36 PM, Peter Collingbourne wrote:
> Hi Tanya,
> On Mon, Nov 29, 2010 at 01:59:17PM -0800, Tanya Lattner wrote:
>> Looks good to me. However, I noticed that this is now dependent upon the language command line args now being parsed before the codegen args. Is this really necessary? I don't think we should have a dependency like that. I'd probably just go ahead and assume the person using the option knows they are compiling CL code.
> The problem is that the default for OpenCL is to enable optimisations
> (see spec s184.108.40.206), so the CodeGen arg parser needs to know the
> language to determine the default optimisation level.
Ah yes, sorry.. read it backwards. :)
> I suppose this reflects a larger architectural issue with frontend's
> arg parser, but I'm not sure what the long term solution to that
> would be.
Yes, this is a bigger issue then. At the very least, there should be a comment or FIXME added if you are going to depend on the options being parsed in a specific order.
More information about the cfe-commits