[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-commits
Subject:    Re: [PATCH] D11182: [OPENMP 4.0] Initial support for 'omp declare reduction' construct.
From:       "Bataev, Alexey via cfe-commits" <cfe-commits () lists ! llvm ! org>
Date:       2015-08-21 3:20:23
Message-ID: BLU436-SMTP253301F82AA56EE3A84F59B85650 () phx ! gbl
[Download RAW message or body]

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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic