[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:       Alexey Bataev via cfe-commits <cfe-commits () lists ! llvm ! org>
Date:       2015-08-21 3:20:31
Message-ID: 82206cda63046dd025f0e4b8f0728463 () localhost ! localdomain
[Download RAW message or body]

ABataev added a subscriber: ABataev.
ABataev added a comment.

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



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]