I will take a look and put up a patch for it. Kelvin
On Fri, Jul 15, 2016 at 2:46 AM, Alexey Bataev <[email protected]> wrote: > Kelvin, > Please look at these messages and fix the tests. Or tell me and I will > fix them > > Best regards, > Alexey Bataev > ============= > Software Engineer > Intel Compiler Team > > 15.07.2016 9:23, Robinson, Paul пишет: > > > >> -----Original Message----- > >> From: Alexey Bataev [mailto:[email protected]] > >> Sent: Thursday, July 14, 2016 7:51 PM > >> To: [email protected] > >> Cc: [email protected]; [email protected]; [email protected]; > >> [email protected]; [email protected]; [email protected]; Robinson, Paul > >> Subject: Re: [PATCH] D22096: [OpenMP] Sema and parsing for 'target > >> parallel for simd' pragma > >> > >> Hi Paul, > >> Could you provide a little bit more info about diagnostic you see? Maybe > >> the tests just need to be fixed > > That would be fine too. Log of the two failing tests attached. > > Thanks, > > --paulr > > > >> Best regards, > >> Alexey Bataev > >> ============= > >> Software Engineer > >> Intel Compiler Team > >> > >> 15.07.2016 2:09, Paul Robinson пишет: > >>> probinson added a subscriber: probinson. > >>> probinson added a comment. > >>> > >>> I'm seeing a different set of diagnostics in two of these tests, > because > >> we default to C+11, which makes them fail for us. Ideally you'd > >> conditionalize the tests on the value of `__cplusplus` (like Charles Li > >> has been doing for a whole lot of the Clang tests). If that's > >> inconvenient for you right now, would you mind if I added `-std=c++03` > to > >> the following tests? It's just these two that need it: > >>> target_parallel_for_simd_collapse_messages.cpp > >>> target_parallel_for_simd_ordered_messages.cpp > >>> > >>> > >>> Repository: > >>> rL LLVM > >>> > >>> https://reviews.llvm.org/D22096 > >>> > >>> > >>> > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
