[PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.

Bataev, Alexey via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 20 20:20:23 PDT 2015


I'll update patch, Michael

Best regards,
Alexey Bataev
=============
Software Engineer
Intel Compiler Team

20.08.2015 18:40, Michael Wong пишет:
> fraggamuffin added a comment.
>
>
>
>
> ================
> Comment at: lib/Parse/ParseDeclCXX.cpp:3011
> @@ -3010,3 +3010,3 @@
>         if (Tok.is(tok::annot_pragma_openmp)) {
> -        ParseOpenMPDeclarativeDirective();
>           continue;
> ----------------
> While testing this patch with the latest trunk for my work on declare target, I kept getting a build error in ParseDeclCXX.cpp as there seems to be no CurAS in scope, this may be because of recent changes. But this line:
>    if (Tok.is(tok::annot_pragma_openmp))
>      return ParseOpenMPDeclarativeDirective(CurAS);
> Seems to work better as
>    if (Tok.is(tok::annot_pragma_openmp))
>      return ParseOpenMPDeclarativeDirective(AS);
>
>
> http://reviews.llvm.org/D11182
>
>
>



More information about the cfe-commits mailing list