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

Michael Wong via cfe-commits cfe-commits at lists.llvm.org
Thu Aug 20 08:40:36 PDT 2015


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