RE: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-11-04 Thread Anastasia Stulova via cfe-commits
Great! Thanks! From: Liu, Yaxun (Sam) [mailto:yaxun@amd.com] Sent: 04 November 2016 13:52 To: Anastasia Stulova; Richard Smith Cc: nd; cfe-commits@lists.llvm.org Subject: RE: r273191 - [OpenCL] Include opencl-c.h by default as a clang module Hi Richard/Anastasia, I agree some tests

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-11-03 Thread Richard Smith via cfe-commits
Hi, This test (test/Headers/opencl-c-header.cl) is *extremely* slow -- it parses the 17KLoC opencl-c.h header 40 times, and on my Debug builds it's the slowest test by far, taking nearly 2 minutes to run *alone*. (For reference, the *entire clang test suite* finishes in 90s on my machine with

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-23 Thread Ismail Donmez via cfe-commits
esday, June 22, 2016 1:52 PM > To: Liu, Yaxun (Sam) <yaxun@amd.com> > Cc: cfe-commits <cfe-commits@lists.llvm.org> > Subject: Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang > module > > Hi, > > On Wed, Jun 22, 2016 at 7:55 PM, Liu, Yaxun (Sa

RE: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-23 Thread Liu, Yaxun (Sam) via cfe-commits
: Wednesday, June 22, 2016 1:52 PM To: Liu, Yaxun (Sam) <yaxun@amd.com> Cc: cfe-commits <cfe-commits@lists.llvm.org> Subject: Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module Hi, On Wed, Jun 22, 2016 at 7:55 PM, Liu, Yaxun (Sam) <yaxun@amd.com>

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-22 Thread Ismail Donmez via cfe-commits
Hi, On Wed, Jun 22, 2016 at 7:55 PM, Liu, Yaxun (Sam) wrote: > The cmake of Cygwin itself does not support llvm. Which cmake did you use on > Cygwin? Thanks. I use native windows cmake and this is the only test that ever failed with this setup. chmod is from native cygwin

RE: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-22 Thread Liu, Yaxun (Sam) via cfe-commits
.llvm.org> Subject: Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module On Wed, Jun 22, 2016 at 5:45 PM, Liu, Yaxun (Sam) <yaxun@amd.com> wrote: > $ "chmod" "u-w" > "C:\cygwin64\home\ismail\src\llvm\dist\tools\clang\test\Headers\Ou

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-22 Thread Ismail Donmez via cfe-commits
On Wed, Jun 22, 2016 at 5:45 PM, Liu, Yaxun (Sam) wrote: > $ "chmod" "u-w" > "C:\cygwin64\home\ismail\src\llvm\dist\tools\clang\test\Headers\Output\opencl-c-header.cl.tmp/*" > # command stderr: > r.cl.tmp/*: invalid mode: 'mp/*' > > the error msg is strange. On my Cygwin I got

RE: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-22 Thread Liu, Yaxun (Sam) via cfe-commits
axun@amd.com> Cc: cfe-commits <cfe-commits@lists.llvm.org> Subject: Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module Hi, On Mon, Jun 20, 2016 at 10:26 PM, Yaxun Liu via cfe-commits <cfe-commits@lists.llvm.org> wrote: > Author: yaxunl > Date: Mon Jun

Re: r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-22 Thread Ismail Donmez via cfe-commits
Hi, On Mon, Jun 20, 2016 at 10:26 PM, Yaxun Liu via cfe-commits wrote: > Author: yaxunl > Date: Mon Jun 20 14:26:00 2016 > New Revision: 273191 > > URL: http://llvm.org/viewvc/llvm-project?rev=273191=rev > Log: > [OpenCL] Include opencl-c.h by default as a clang

r273191 - [OpenCL] Include opencl-c.h by default as a clang module

2016-06-20 Thread Yaxun Liu via cfe-commits
Author: yaxunl Date: Mon Jun 20 14:26:00 2016 New Revision: 273191 URL: http://llvm.org/viewvc/llvm-project?rev=273191=rev Log: [OpenCL] Include opencl-c.h by default as a clang module Include opencl-c.h by default as a module to utilize the automatic AST caching mechanism of clang modules.