r202360 - [OPENMP] First changes for Parsing and Sema for 'omp simd' directive support

Alp Toker alp at nuanti.com
Thu Feb 27 07:14:01 PST 2014


On 27/02/2014 14:49, Aaron Ballman wrote:
> On Thu, Feb 27, 2014 at 9:44 AM, Alexander Musman
> <alexander.musman at gmail.com> wrote:
>> Hi Aaron,
>>
>> This warning is due to 'omp simd' had no allowed clauses in the first patch.
>> I hope to fix this soon by adding clauses (I will send first clause for
>> review tomorrow or early next week).
> We usually like to keep the builds warning-free whenever possible, but
> since this is an expected behavior and work is being done soon to
> rectify it, I think it's fine to leave in. I mostly just wanted to
> make sure this wasn't an unexpected bug (as has happened before).
>
> However, if you think it'll take longer than early next week to
> rectify this, it might make sense to remove the basically empty switch
> until you've got some content for it.

It would be nice for review as well to add a use along with new 
facilities were possible in future to give an idea of what they're for 
(this applies for the proposed -openmp= flag patch too).

Alp.


>
> Thanks!
>
> ~Aaron
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list