<div dir="ltr"><div>I will take a look and put up a patch for it.<br><br></div>Kelvin<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 15, 2016 at 2:46 AM, Alexey Bataev <span dir="ltr"><<a href="mailto:a.bataev@hotmail.com" target="_blank">a.bataev@hotmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Kelvin,<br>
Please look at these messages and fix the tests. Or tell me and I will<br>
fix them<br>
<br>
Best regards,<br>
Alexey Bataev<br>
=============<br>
Software Engineer<br>
Intel Compiler Team<br>
<br>
15.07.2016 9:23, Robinson, Paul пишет:<br>
><br>
>> -----Original Message-----<br>
>> From: Alexey Bataev [mailto:<a href="mailto:a.bataev@hotmail.com">a.bataev@hotmail.com</a>]<br>
>> Sent: Thursday, July 14, 2016 7:51 PM<br>
>> To: <a href="mailto:reviews%2Bd22096%2Bpublic%2B4c00789034d62932@reviews.llvm.org">reviews+d22096+public+4c00789034d62932@reviews.llvm.org</a><br>
>> Cc: <a href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a>; <a href="mailto:kkwli0@gmail.com">kkwli0@gmail.com</a>; <a href="mailto:cbertol@us.ibm.com">cbertol@us.ibm.com</a>;<br>
>> <a href="mailto:sfantao@us.ibm.com">sfantao@us.ibm.com</a>; <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>; <a href="mailto:acjacob@us.ibm.com">acjacob@us.ibm.com</a>; Robinson, Paul<br>
>> Subject: Re: [PATCH] D22096: [OpenMP] Sema and parsing for 'target<br>
>> parallel for simd' pragma<br>
>><br>
>> Hi Paul,<br>
>> Could you provide a little bit more info about diagnostic you see? Maybe<br>
>> the tests just need to be fixed<br>
> That would be fine too.  Log of the two failing tests attached.<br>
> Thanks,<br>
> --paulr<br>
><br>
>> Best regards,<br>
>> Alexey Bataev<br>
>> =============<br>
>> Software Engineer<br>
>> Intel Compiler Team<br>
>><br>
>> 15.07.2016 2:09, Paul Robinson пишет:<br>
>>> probinson added a subscriber: probinson.<br>
>>> probinson added a comment.<br>
>>><br>
>>> I'm seeing a different set of diagnostics in two of these tests, because<br>
>> we default to C+11, which makes them fail for us.  Ideally you'd<br>
>> conditionalize the tests on the value of `__cplusplus` (like Charles Li<br>
>> has been doing for a whole lot of the Clang tests).  If that's<br>
>> inconvenient for you right now, would you mind if I added `-std=c++03` to<br>
>> the following tests?  It's just these two that need it:<br>
>>> target_parallel_for_simd_collapse_messages.cpp<br>
>>> target_parallel_for_simd_ordered_messages.cpp<br>
>>><br>
>>><br>
>>> Repository:<br>
>>>     rL LLVM<br>
>>><br>
>>> <a href="https://reviews.llvm.org/D22096" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22096</a><br>
>>><br>
>>><br>
>>><br>
<br>
</blockquote></div><br></div>