Hi, I won't have access to the system for ~2 weeks but the change looks fine. Please commit if it passes the test on Linux with that.
Regards, ismail On Thu, Jun 23, 2016 at 4:39 PM, Liu, Yaxun (Sam) <yaxun....@amd.com> wrote: > I have a patch which may workaround this issue, however I could not test it > in Cygwin since I got issues build latest cmake in Cygwin and the downloaded > cmake does not work. > > Could you please try it? > > Thanks. > > Sam > > -----Original Message----- > From: Ismail Donmez [mailto:ism...@i10z.com] > Sent: 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> 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 which seems to be the problem. > > ismail _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits