Re: RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-03 Thread Samuel F Antao via cfe-commits
@lists.llvm.orgSubject: RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.   Hi Samuel,   if we trust StackOverflow, the code is wrong because all functions with the same name are hidden, ignoring the rest of the signature. I will try to find the corresponding se

Re: RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-03 Thread Samuel F Antao via cfe-commits
rg] On Behalf Of Hahnfeld, Jonas via cfe-commitsSent: Thursday, November 03, 2016 1:11 PMTo: Samuel F AntaoCc: cfe-commits@lists.llvm.orgSubject: RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.   Hi Samuel,   if we trust StackOverflow, the code is wrong because all

RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-03 Thread Hahnfeld, Jonas via cfe-commits
ursday, November 03, 2016 1:04 PM To: Hahnfeld, Jonas Cc: cfe-commits@lists.llvm.org; gkistan...@gmail.com Subject: Re: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions. Hi Jonas, I think that the code is correct. Is probably some bug in the compiler. We

Re: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-03 Thread Samuel F Antao via cfe-commits
e-commitsSubject: Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.   http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/128/steps/cmake-configure/logs/stdio-- The C compiler identification is GNU 4.8.4-- The CXX compiler identification is GNU 4.8.4The code it

RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-03 Thread Hahnfeld, Jonas via cfe-commits
- [Driver][OpenMP] Add support to create jobs for unbundling actions. http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/128/steps/cmake-configure/logs/stdio -- The C compiler identification is GNU 4.8.4 -- The CXX compiler identification is GNU 4.8.4 The code itself seems fine. The

Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-11-01 Thread Galina Kistanova via cfe-commits
achable > statement, so no need for return. So, the code seems to be okay, but we can > probably massage the code a little to silence the compiler you are using. > > Thanks! > Samuel > > > - Original message - > From: Galina Kistanova > To: Samuel F Antao/W

Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-10-31 Thread Samuel F Antao via cfe-commits
Hi Galina,   Thanks for letting me know. Can you tell me which compiler (kind and version) is used in the buildbot slave? Looks like that compiler is not doing what it should, so I need to be able to test a workaround.   We have     virtual void ConstructJob(Compilation &C, const JobAction &JA,   

Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-10-31 Thread Galina Kistanova via cfe-commits
Hi Samuel, Looks like this revision introduced warning to one of our builders: http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/67/steps/build-stage3-clang/logs/warnings%20%28830%29 Please have a look at this? Thanks Galina On Thu, Oct 27, 2016 at 11:14 AM, Samuel Antao via cfe-com

r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.

2016-10-27 Thread Samuel Antao via cfe-commits
Author: sfantao Date: Thu Oct 27 13:14:55 2016 New Revision: 285326 URL: http://llvm.org/viewvc/llvm-project?rev=285326&view=rev Log: [Driver][OpenMP] Add support to create jobs for unbundling actions. Summary: This patch adds the support to create jobs for the `OffloadBundlingAction` which will