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

[Bataev, Alexey via cfe-commits]

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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits