Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-29 Thread Hans Wennborg via cfe-commits
is possible at all. > > Thanks again, > Samuel > > > - Original message - > From: Hans Wennborg > Sent by: hwennb...@google.com > To: Samuel F Antao/Watson/IBM@IBMUS > Cc: Mike Aizatsky , cfe-commits > , "Bataev, Alexey" > Subject: Re: r276977 - [O

Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-29 Thread Samuel F Antao via cfe-commits
ng any> non-portable code pattern here.>> Alexey, do you have a easy way to test this? Can you give me a little help> here?>> Thanks!> Samuel>>> - Original message -> From: Hans Wennborg > Sent by: hwennb...@google.com> To: Samuel F Antao/Watson/IBM@

Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-29 Thread Hans Wennborg via cfe-commits
y bot message >> complaining >> about that code. >> >> Do you still see the issues? >> >> Thanks, >> Samuel >> >> >> - Original message - >> From: Mike Aizatsky >> To: Samuel F Antao/Watson/IBM@IBMUS, cfe-commits@lis

Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-29 Thread Samuel F Antao via cfe-commits
shortly after> I pushed the patch. After that I didn't receive any bot message complaining> about that code.>> Do you still see the issues?>> Thanks,> Samuel>>> - Original message -> From: Mike Aizatsky > To: Samuel F Antao/Watson/IBM@IBMUS, cfe-commits

Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-29 Thread Hans Wennborg via cfe-commits
cfe-commits@lists.llvm.org > Cc: > Subject: Re: r276977 - [OpenMP] Codegen for use_device_ptr clause. > Date: Thu, Jul 28, 2016 4:20 PM > > Samuel, > > I think your change breaks the build: > > http://lab.llvm.org:8011/builders/sanitizer-windows/builds/26397 > > F

Re: r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-28 Thread Samuel F Antao via cfe-commits
Hi Mike,   I've already pushed r276981 and r276988 to fix those failures shortly after I pushed the patch. After that I didn't receive any bot message complaining about that code.    Do you still see the issues?   Thanks, Samuel   - Original message -From: Mike Aizatsky To: Samuel F Antao/W

r276977 - [OpenMP] Codegen for use_device_ptr clause.

2016-07-28 Thread Samuel Antao via cfe-commits
Author: sfantao Date: Thu Jul 28 09:23:26 2016 New Revision: 276977 URL: http://llvm.org/viewvc/llvm-project?rev=276977&view=rev Log: [OpenMP] Codegen for use_device_ptr clause. Summary: This patch adds support for the use_device_ptr clause. It includes changes in SEMA that could not be tested w