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

2016-11-03 Thread Samuel F Antao via cfe-commits
  ^~~~   Regards, Jonas   From: Samuel F Antao [mailto:sfan...@us.ibm.com]Sent: Thursday, November 03, 2016 1:04 PMTo: Hahnfeld, JonasCc: cfe-commits@lists.llvm.org; gkistan...@gmail.comSubject: Re: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.   Hi Jonas,   I th

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