<div class="socmaildefaultfont" dir="ltr" style="font-family:Arial, Helvetica, sans-serif;font-size:10.5pt" ><div dir="ltr" >Hi Jonas,</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks for the quotes. I'll rename one of the versions of ConstructJob to make the problem disappear. I'll post a patch soon.</div>
<div dir="ltr" > </div>
<div dir="ltr" >Thanks again,</div>
<div dir="ltr" >Samuel </div>
<div dir="ltr" > </div>
<blockquote data-history-content-modified="1" dir="ltr" style="border-left:solid #aaaaaa 2px; margin-left:5px; padding-left:5px; direction:ltr; margin-right:0px" >----- Original message -----<br>From: "Hahnfeld, Jonas" <Hahnfeld@itc.rwth-aachen.de><br>To: "Hahnfeld, Jonas" <Hahnfeld@itc.rwth-aachen.de>, Samuel F Antao/Watson/IBM@IBMUS<br>Cc: "cfe-commits@lists.llvm.org" <cfe-commits@lists.llvm.org><br>Subject: RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.<br>Date: Thu, Nov 3, 2016 12:36 PM<br> <br><!--Notes ACF
<meta http-equiv=Content-Type content="text/html; charset=utf8" >--><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit" >
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
<div><p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >From C++14:<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >3.4.1 Unqualified name lookup, (1):<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >“In all the cases listed in 3.4.1, the scopes are searched for a declaration in the order listed in each of the respective categories; name lookup ends as soon as a declaration is found for the name. If no declaration is found, the program is ill-formed.”<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >I assume that “the order listed in each of the respective categories” for method lookup means “from derived to base classes” (though I haven’t looked this up…)<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >3.4 Name lookup, (1):<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >“[…]Only after name lookup, function overload resolution (if applicable) and access checking have succeeded are the attributes introduced by the name’s declaration used further in expression processing (Clause 5). […]”<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >I think this means that the compiler will for example find clang::driver::tools::Clang::ConstructJob but is not allowed to use clang::driver::Tool::ConstructJob as a resolution unless “using Tool::ConstructJob;” is specified.<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Cheers,<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Jonas<o:p></o:p></span></p>
<p><o:p> </o:p></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt" ><div><div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm" ><p><b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" >From:</span></b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" > cfe-commits [mailto:cfe-commits-bounces@lists.llvm.org] <b>On Behalf Of </b>Hahnfeld, Jonas via cfe-commits<br><b>Sent:</b> Thursday, November 03, 2016 1:11 PM<br><b>To:</b> Samuel F Antao<br><b>Cc:</b> cfe-commits@lists.llvm.org<br><b>Subject:</b> RE: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.<o:p></o:p></span></p></div></div>
<p><o:p> </o:p></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Hi Samuel,<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >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 section in the C++ standard…<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >I think any version of GCC gives the error, I was able to reproduce this with the attached small code:<br>$ g++ --version<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >g++ (GCC) 6.2.0<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >[…]<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >$ g++ -Woverloaded-virtual virtual-name-hiding.cpp -o virtual-name-hiding.exe<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >virtual-name-hiding.cpp:2:16: warning: ‘virtual void Base::test(const long int&)’ was hidden [-Woverloaded-virtual]<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >   virtual void test(const long &l);<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >                ^~~~<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >virtual-name-hiding.cpp:7:16: warning:   by ‘virtual void Child::test(const int&)’ [-Woverloaded-virtual]<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >   virtual void test(const int &a) override { };<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >                ^~~~<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" ><o:p> </o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Regards,<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Jonas<o:p></o:p></span></p>
<p><o:p> </o:p></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt" ><div><div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm" ><p><b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" >From:</span></b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" > Samuel F Antao [<a href="mailto:sfantao@us.ibm.com" target="_blank" >mailto:sfantao@us.ibm.com</a>]<br><b>Sent:</b> Thursday, November 03, 2016 1:04 PM<br><b>To:</b> Hahnfeld, Jonas<br><b>Cc:</b> <a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a>; <a href="mailto:gkistanova@gmail.com" target="_blank" >gkistanova@gmail.com</a><br><b>Subject:</b> Re: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.<o:p></o:p></span></p></div></div>
<p><o:p> </o:p></p>
<div><div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >Hi Jonas,<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >I think that the code is correct. Is probably some bug in the compiler. We are not hiding anything - the compiler is checking against the wrong virtual function (using the wrong signature).<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >I am in process to compile the same version of gcc the bot is using to see if there is any massagin in the code that can be done to silence that.<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >Thanks!<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >Samuel<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<blockquote style="border:none;border-left:solid #AAAAAA 1.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt" ><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >----- Original message -----<br>From: "Hahnfeld, Jonas" <<a href="mailto:Hahnfeld@itc.rwth-aachen.de" target="_blank" >Hahnfeld@itc.rwth-aachen.de</a>><br>To: Galina Kistanova <<a href="mailto:gkistanova@gmail.com" target="_blank" >gkistanova@gmail.com</a>>, Samuel F Antao/Watson/IBM@IBMUS<br>Cc: "<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a>" <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a>><br>Subject: RE: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.<br>Date: Thu, Nov 3, 2016 11:53 AM<br> <o:p></o:p></span></p>
<div><p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Hi,</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >I think I have identified what is the problem here: the so-called “name hiding” when overloading virtual functions. (see <a href="http://stackoverflow.com/a/1629074" target="_blank" >http://stackoverflow.com/a/1629074</a></span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >)<o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >To fix it, I see two possible solutions: Either apply the attached patch or rename one method so that the hiding does not happen.</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Finally: Why didn’t the other bots point to this problem?!?</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Cheers,</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D" >Jonas</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt" ><div><div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm" ><p><b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" >From:</span></b><span lang="DE" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"" > cfe-commits [<a href="mailto:cfe-commits-bounces@lists.llvm.org" target="_blank" >mailto:cfe-commits-bounces@lists.llvm.org</a>] <b>On Behalf Of </b>Galina Kistanova via cfe-commits<br><b>Sent:</b> Tuesday, November 01, 2016 7:29 PM<br><b>To:</b> Samuel F Antao<br><b>Cc:</b> cfe-commits<br><b>Subject:</b> Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div></div>
<p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p>
<div><p style="margin-bottom:12.0pt" ><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><a href="http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/128/steps/cmake-configure/logs/stdio" target="_blank" >http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/128/steps/cmake-configure/logs/stdio</a></span><br><br>-- The C compiler identification is GNU 4.8.4<br>-- The CXX compiler identification is GNU 4.8.4<br><br>The code itself seems fine. The similar builder that uses the ToT Clang does not generate any warnings.<br><a href="http://lab.llvm.org:8011/builders/clang-with-lto-ubuntu/builds/107" target="_blank" >http://lab.llvm.org:8011/builders/clang-with-lto-ubuntu/builds/107</a>​<br><br>Thanks<br><br>Galina<o:p></o:p></p>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >On Mon, Oct 31, 2016 at 6:26 PM, Samuel F Antao <<a href="mailto:sfantao@us.ibm.com" target="_blank" >sfantao@us.ibm.com</a>> wrote:<o:p></o:p></span></p>
<div><div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >Hi Galina,</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >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.</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >We have  </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >virtual void ConstructJob(Compilation &C, const JobAction &JA,<br>                            const InputInfo &Output,<br>                            const InputInfoList &Inputs,<br>                            const llvm::opt::ArgList &TCArgs,<br>                            const char *LinkingOutput) const = 0;</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >which is the signature of ConstructJob that should be overwritten, so we are not hiding anything. And in Action.cpp we have an unreachable 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.</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >Thanks!</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue";color:#888888" >Samuel</span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<div><div><div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div>
<blockquote style="border:none;border-left:solid #AAAAAA 1.5pt;padding:0cm 0cm 0cm 4.0pt;margin-left:3.75pt;margin-top:5.0pt;margin-right:0cm;margin-bottom:5.0pt" ><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >----- Original message -----<br>From: Galina Kistanova <<a href="mailto:gkistanova@gmail.com" target="_blank" >gkistanova@gmail.com</a>><br>To: Samuel F Antao/Watson/IBM@IBMUS<br>Cc: cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a>><br>Subject: Re: r285326 - [Driver][OpenMP] Add support to create jobs for unbundling actions.<br>Date: Tue, Nov 1, 2016 12:59 AM<br>  </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >Hi Samuel,<br><br>Looks like this revision introduced warning to one of our builders:<br><a href="http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/67/steps/build-stage3-clang/logs/warnings%20%28830%29" target="_blank" >http://lab.llvm.org:8011/builders/clang-3stage-ubuntu/builds/67/steps/build-stage3-clang/logs/warnings%20%28830%29</a></span><br><br><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >Please have a look at this?<br><br>Thanks<br><br>Galina<o:p></o:p></span></p></div>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >  </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >On Thu, Oct 27, 2016 at 11:14 AM, Samuel Antao via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a>> wrote: </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p>
<p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" >Author: sfantao<br>Date: Thu Oct 27 13:14:55 2016<br>New Revision: 285326<br><br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a></span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" >-project?rev=285326&view=rev<br>Log:<br>[Driver][OpenMP] Add support to create jobs for unbundling actions.<br><br>Summary:<br>This patch adds the support to create jobs for the `OffloadBundlingAction` which will invoke the `clang-offload-bundler` tool to unbundle input files.<br><br>Unlike other actions, unbundling actions have multiple outputs. Therefore, this patch adds the required changes to have a variant of `Tool::ConstructJob` with multiple outputs.<br><br>The way the naming of the results is implemented is also slightly modified so that the same action can use a different offloading prefix for each use by the different offloading actions.<br><br>With this patch, it is possible to compile a functional OpenMP binary with offloading support, even with separate compilation.<br><br>Reviewers: echristo, tra, jlebar, ABataev, hfinkel<br><br>Subscribers: mkuron, whchung, mehdi_amini, cfe-commits, Hahnfeld, andreybokhanko, arpith-jacob, carlo.bertolli, caomhin<br><br>Differential Revision: <a href="https://reviews.llvm.org/D21857" target="_blank" >https://reviews.llvm.org/D21857</a><br><br>Modified:<br>    cfe/trunk/include/clang/Driver/Action.h<br>    cfe/trunk/include/clang/Driver/Driver.h<br>    cfe/trunk/include/clang/Driver/Tool.h<br>    cfe/trunk/lib/Driver/Action.cpp<br>    cfe/trunk/lib/Driver/Driver.cpp<br>    cfe/trunk/lib/Driver/Tool.cpp<br>    cfe/trunk/lib/Driver/Tools.cpp<br>    cfe/trunk/lib/Driver/Tools.h<br>    cfe/trunk/test/Driver/<a href="http://cuda-bindings.cu" target="_blank" >cuda-bindings.cu</a><br>    cfe/trunk/test/Driver/openmp-offload.c<br>    cfe/trunk/test/Driver/opt-record.c<br><br>Modified: cfe/trunk/include/clang/Driver/Action.h<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a></span>-project/cfe/trunk/include/clang/Driver/Action.h?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/include/clang/Driver/Action.h (original)<br>+++ cfe/trunk/include/clang/Driver/Action.h Thu Oct 27 13:14:55 2016<br>@@ -157,9 +157,12 @@ public:<br>   /// Return a string containing the offload kind of the action.<br>   std::string getOffloadingKindPrefix() const;<br>   /// Return a string that can be used as prefix in order to generate unique<br>-  /// files for each offloading kind.<br>-  std::string<br>-  getOffloadingFileNamePrefix(llvm::StringRef NormalizedTriple) const;<br>+  /// files for each offloading kind. By default, no prefix is used for<br>+  /// non-device kinds, except if \a CreatePrefixForHost is set.<br>+  static std::string<br>+  GetOffloadingFileNamePrefix(OffloadKind Kind,<br>+                              llvm::StringRef NormalizedTriple,<br>+                              bool CreatePrefixForHost = false);<br>   /// Return a string containing a offload kind name.<br>   static StringRef GetOffloadKindName(OffloadKind Kind);<br><br><br>Modified: cfe/trunk/include/clang/Driver/Driver.h<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/include/clang/Driver/Driver.h?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/include/clang/Driver/Driver.h (original)<br>+++ cfe/trunk/include/clang/Driver/Driver.h Thu Oct 27 13:14:55 2016<br>@@ -12,6 +12,7 @@<br><br> #include "clang/Basic/Diagnostic.h"<br> #include "clang/Basic/LLVM.h"<br>+#include "clang/Driver/Action.h"<br> #include "clang/Driver/Phases.h"<br> #include "clang/Driver/Types.h"<br> #include "clang/Driver/Util.h"<br>@@ -42,7 +43,6 @@ class FileSystem;<br><br> namespace driver {<br><br>-  class Action;<br>   class Command;<br>   class Compilation;<br>   class InputInfo;<br>@@ -417,14 +417,14 @@ public:<br><br>   /// BuildJobsForAction - Construct the jobs to perform for the action \p A and<br>   /// return an InputInfo for the result of running \p A.  Will only construct<br>-  /// jobs for a given (Action, ToolChain, BoundArch) tuple once.<br>+  /// jobs for a given (Action, ToolChain, BoundArch, DeviceKind) tuple once.<br>   InputInfo<br>   BuildJobsForAction(Compilation &C, const Action *A, const ToolChain *TC,<br>                      StringRef BoundArch, bool AtTopLevel, bool MultipleArchs,<br>                      const char *LinkingOutput,<br>                      std::map<std::pair<const Action *, std::string>, InputInfo><br>                          &CachedResults,<br>-                     bool BuildForOffloadDevice) const;<br>+                     Action::OffloadKind TargetDeviceOffloadKind) const;<br><br>   /// Returns the default name for linked images (e.g., "a.out").<br>   const char *getDefaultImageName() const;<br>@@ -495,7 +495,7 @@ private:<br>       bool AtTopLevel, bool MultipleArchs, const char *LinkingOutput,<br>       std::map<std::pair<const Action *, std::string>, InputInfo><br>           &CachedResults,<br>-      bool BuildForOffloadDevice) const;<br>+      Action::OffloadKind TargetDeviceOffloadKind) const;<br><br> public:<br>   /// GetReleaseVersion - Parse (([0-9]+)(.([0-9]+)(.([0-9]+)?))?)? and<br><br>Modified: cfe/trunk/include/clang/Driver/Tool.h<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/include/clang/Driver/Tool.h?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/include/clang/Driver/Tool.h (original)<br>+++ cfe/trunk/include/clang/Driver/Tool.h Thu Oct 27 13:14:55 2016<br>@@ -129,6 +129,20 @@ public:<br>                             const InputInfoList &Inputs,<br>                             const llvm::opt::ArgList &TCArgs,<br>                             const char *LinkingOutput) const = 0;<br>+  /// Construct jobs to perform the action \p JA, writing to the \p Outputs and<br>+  /// with \p Inputs, and add the jobs to \p C. The default implementation<br>+  /// assumes a single output and is expected to be overloaded for the tools<br>+  /// that support multiple inputs.<br>+  ///<br>+  /// \param TCArgs The argument list for this toolchain, with any<br>+  /// tool chain specific translations applied.<br>+  /// \param LinkingOutput If this output will eventually feed the<br>+  /// linker, then this is the final output name of the linked image.<br>+  virtual void ConstructJob(Compilation &C, const JobAction &JA,<br>+                            const InputInfoList &Outputs,<br>+                            const InputInfoList &Inputs,<br>+                            const llvm::opt::ArgList &TCArgs,<br>+                            const char *LinkingOutput) const;<br> };<br><br> } // end namespace driver<br><br>Modified: cfe/trunk/lib/Driver/Action.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/lib/Driver/Action.cpp?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/lib/Driver/Action.cpp (original)<br>+++ cfe/trunk/lib/Driver/Action.cpp Thu Oct 27 13:14:55 2016<br>@@ -115,15 +115,18 @@ std::string Action::getOffloadingKindPre<br>   return Res;<br> }<br><br>+/// Return a string that can be used as prefix in order to generate unique files<br>+/// for each offloading kind.<br> std::string<br>-Action::getOffloadingFileNamePrefix(llvm::StringRef NormalizedTriple) const {<br>-  // A file prefix is only generated for device actions and consists of the<br>-  // offload kind and triple.<br>-  if (!OffloadingDeviceKind)<br>+Action::GetOffloadingFileNamePrefix(OffloadKind Kind,<br>+                                    llvm::StringRef NormalizedTriple,<br>+                                    bool CreatePrefixForHost) {<br>+  // Don't generate prefix for host actions unless required.<br>+  if (!CreatePrefixForHost && (Kind == OFK_None || Kind == OFK_Host))<br>     return "";<br><br>   std::string Res("-");<br>-  Res += getOffloadingKindPrefix();<br>+  Res += GetOffloadKindName(Kind);<br>   Res += "-";<br>   Res += NormalizedTriple;<br>   return Res;<br><br>Modified: cfe/trunk/lib/Driver/Driver.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/lib/Driver/Driver.cpp?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/lib/Driver/Driver.cpp (original)<br>+++ cfe/trunk/lib/Driver/Driver.cpp Thu Oct 27 13:14:55 2016<br>@@ -2593,7 +2593,7 @@ void Driver::BuildJobs(Compilation &C) c<br>                        /*AtTopLevel*/ true,<br>                        /*MultipleArchs*/ ArchNames.size() > 1,<br>                        /*LinkingOutput*/ LinkingOutput, CachedResults,<br>-                       /*BuildForOffloadDevice*/ false);<br>+                       /*TargetDeviceOffloadKind*/ Action::OFK_None);<br>   }<br><br>   // If the user passed -Qunused-arguments or there were errors, don't warn<br>@@ -2926,31 +2926,38 @@ public:<br> };<br> }<br><br>-InputInfo Driver::BuildJobsForAction(<br>-    Compilation &C, const Action *A, const ToolChain *TC, StringRef BoundArch,<br>-    bool AtTopLevel, bool MultipleArchs, const char *LinkingOutput,<br>-    std::map<std::pair<const Action *, std::string>, InputInfo> &CachedResults,<br>-    bool BuildForOffloadDevice) const {<br>-  // The bound arch is not necessarily represented in the toolchain's triple --<br>-  // for example, armv7 and armv7s both map to the same triple -- so we need<br>-  // both in our map. Also, we need to add the offloading device kind, as the<br>-  // same tool chain can be used for host and device for some programming<br>-  // models, e.g. OpenMP.<br>+/// Return a string that uniquely identifies the result of a job. The bound arch<br>+/// is not necessarily represented in the toolchain's triple -- for example,<br>+/// armv7 and armv7s both map to the same triple -- so we need both in our map.<br>+/// Also, we need to add the offloading device kind, as the same tool chain can<br>+/// be used for host and device for some programming models, e.g. OpenMP.<br>+static std::string GetTriplePlusArchString(const ToolChain *TC,<br>+                                           StringRef BoundArch,<br>+                                           Action::OffloadKind OffloadKind) {<br>   std::string TriplePlusArch = TC->getTriple().normalize();<br>   if (!BoundArch.empty()) {<br>     TriplePlusArch += "-";<br>     TriplePlusArch += BoundArch;<br>   }<br>   TriplePlusArch += "-";<br>-  TriplePlusArch += A->getOffloadingKindPrefix();<br>-  std::pair<const Action *, std::string> ActionTC = {A, TriplePlusArch};<br>+  TriplePlusArch += Action::GetOffloadKindName(OffloadKind);<br>+  return TriplePlusArch;<br>+}<br>+<br>+InputInfo Driver::BuildJobsForAction(<br>+    Compilation &C, const Action *A, const ToolChain *TC, StringRef BoundArch,<br>+    bool AtTopLevel, bool MultipleArchs, const char *LinkingOutput,<br>+    std::map<std::pair<const Action *, std::string>, InputInfo> &CachedResults,<br>+    Action::OffloadKind TargetDeviceOffloadKind) const {<br>+  std::pair<const Action *, std::string> ActionTC = {<br>+      A, GetTriplePlusArchString(TC, BoundArch, TargetDeviceOffloadKind)};<br>   auto CachedResult = CachedResults.find(ActionTC);<br>   if (CachedResult != CachedResults.end()) {<br>     return CachedResult->second;<br>   }<br>   InputInfo Result = BuildJobsForActionNoCache(<br>       C, A, TC, BoundArch, AtTopLevel, MultipleArchs, LinkingOutput,<br>-      CachedResults, BuildForOffloadDevice);<br>+      CachedResults, TargetDeviceOffloadKind);<br>   CachedResults[ActionTC] = Result;<br>   return Result;<br> }<br>@@ -2959,10 +2966,11 @@ InputInfo Driver::BuildJobsForActionNoCa<br>     Compilation &C, const Action *A, const ToolChain *TC, StringRef BoundArch,<br>     bool AtTopLevel, bool MultipleArchs, const char *LinkingOutput,<br>     std::map<std::pair<const Action *, std::string>, InputInfo> &CachedResults,<br>-    bool BuildForOffloadDevice) const {<br>+    Action::OffloadKind TargetDeviceOffloadKind) const {<br>   llvm::PrettyStackTraceString CrashInfo("Building compilation jobs");<br><br>   InputInfoList OffloadDependencesInputInfo;<br>+  bool BuildingForOffloadDevice = TargetDeviceOffloadKind != Action::OFK_None;<br>   if (const OffloadAction *OA = dyn_cast<OffloadAction>(A)) {<br>     // The offload action is expected to be used in four different situations.<br>     //<br>@@ -2995,7 +3003,7 @@ InputInfo Driver::BuildJobsForActionNoCa<br>         DevA =<br>             BuildJobsForAction(C, DepA, DepTC, DepBoundArch, AtTopLevel,<br>                                /*MultipleArchs*/ !!DepBoundArch, LinkingOutput,<br>-                               CachedResults, /*BuildForOffloadDevice=*/true);<br>+                               CachedResults, DepA->getOffloadingDeviceKind());<br>       });<br>       return DevA;<br>     }<br>@@ -3005,16 +3013,15 @@ InputInfo Driver::BuildJobsForActionNoCa<br>     // generate the host dependences and override the action with the device<br>     // dependence. The dependences can't therefore be a top-level action.<br>     OA->doOnEachDependence(<br>-        /*IsHostDependence=*/BuildForOffloadDevice,<br>+        /*IsHostDependence=*/BuildingForOffloadDevice,<br>         [&](Action *DepA, const ToolChain *DepTC, const char *DepBoundArch) {<br>           OffloadDependencesInputInfo.push_back(BuildJobsForAction(<br>               C, DepA, DepTC, DepBoundArch, /*AtTopLevel=*/false,<br>               /*MultipleArchs*/ !!DepBoundArch, LinkingOutput, CachedResults,<br>-              /*BuildForOffloadDevice=*/DepA->getOffloadingDeviceKind() !=<br>-                  Action::OFK_None));<br>+              DepA->getOffloadingDeviceKind()));<br>         });<br><br>-    A = BuildForOffloadDevice<br>+    A = BuildingForOffloadDevice<br>             ? OA->getSingleDeviceDependence(/*DoNotConsiderHostActions=*/true)<br>             : OA->getHostDependence();<br>   }<br>@@ -3044,7 +3051,7 @@ InputInfo Driver::BuildJobsForActionNoCa<br><br>     return BuildJobsForAction(C, *BAA->input_begin(), TC, ArchName, AtTopLevel,<br>                               MultipleArchs, LinkingOutput, CachedResults,<br>-                              BuildForOffloadDevice);<br>+                              TargetDeviceOffloadKind);<br>   }<br><br><br>@@ -3063,13 +3070,12 @@ InputInfo Driver::BuildJobsForActionNoCa<br>   // need to build jobs for host/device-side inputs it may have held.<br>   for (const auto *OA : CollapsedOffloadActions)<br>     cast<OffloadAction>(OA)->doOnEachDependence(<br>-        /*IsHostDependence=*/BuildForOffloadDevice,<br>+        /*IsHostDependence=*/BuildingForOffloadDevice,<br>         [&](Action *DepA, const ToolChain *DepTC, const char *DepBoundArch) {<br>           OffloadDependencesInputInfo.push_back(BuildJobsForAction(<br>               C, DepA, DepTC, DepBoundArch, /* AtTopLevel */ false,<br>               /*MultipleArchs=*/!!DepBoundArch, LinkingOutput, CachedResults,<br>-              /*BuildForOffloadDevice=*/DepA->getOffloadingDeviceKind() !=<br>-                  Action::OFK_None));<br>+              DepA->getOffloadingDeviceKind()));<br>         });<br><br>   // Only use pipes when there is exactly one input.<br>@@ -3082,7 +3088,7 @@ InputInfo Driver::BuildJobsForActionNoCa<br>         AtTopLevel && (isa<DsymutilJobAction>(A) || isa<VerifyJobAction>(A));<br>     InputInfos.push_back(BuildJobsForAction(<br>         C, Input, TC, BoundArch, SubJobAtTopLevel, MultipleArchs, LinkingOutput,<br>-        CachedResults, BuildForOffloadDevice));<br>+        CachedResults, A->getOffloadingDeviceKind()));<br>   }<br><br>   // Always use the first input as the base input.<br>@@ -3114,13 +3120,59 @@ InputInfo Driver::BuildJobsForActionNoCa<br><br>   // Determine the place to write output to, if any.<br>   InputInfo Result;<br>-  if (JA->getType() == types::TY_Nothing)<br>+  InputInfoList UnbundlingResults;<br>+  if (auto *UA = dyn_cast<OffloadUnbundlingJobAction>(JA)) {<br>+    // If we have an unbundling job, we need to create results for all the<br>+    // outputs. We also update the results cache so that other actions using<br>+    // this unbundling action can get the right results.<br>+    for (auto &UI : UA->getDependentActionsInfo()) {<br>+      assert(UI.DependentOffloadKind != Action::OFK_None &&<br>+             "Unbundling with no offloading??");<br>+<br>+      // Unbundling actions are never at the top level. When we generate the<br>+      // offloading prefix, we also do that for the host file because the<br>+      // unbundling action does not change the type of the output which can<br>+      // cause a overwrite.<br>+      std::string OffloadingPrefix = Action::GetOffloadingFileNamePrefix(<br>+          UI.DependentOffloadKind,<br>+          UI.DependentToolChain->getTriple().normalize(),<br>+          /*CreatePrefixForHost=*/true);<br>+      auto CurI = InputInfo(<br>+          UA, GetNamedOutputPath(C, *UA, BaseInput, UI.DependentBoundArch,<br>+                                 /*AtTopLevel=*/false, MultipleArchs,<br>+                                 OffloadingPrefix),<br>+          BaseInput);<br>+      // Save the unbundling result.<br>+      UnbundlingResults.push_back(CurI);<br>+<br>+      // Get the unique string identifier for this dependence and cache the<br>+      // result.<br>+      CachedResults[{A, GetTriplePlusArchString(<br>+                            UI.DependentToolChain, UI.DependentBoundArch,<br>+                            UI.DependentOffloadKind)}] = CurI;<br>+    }<br>+<br>+    // Now that we have all the results generated, select the one that should be<br>+    // returned for the current depending action.<br>+    std::pair<const Action *, std::string> ActionTC = {<br>+        A, GetTriplePlusArchString(TC, BoundArch, TargetDeviceOffloadKind)};<br>+    assert(CachedResults.find(ActionTC) != CachedResults.end() &&<br>+           "Result does not exist??");<br>+    Result = CachedResults[ActionTC];<br>+  } else if (JA->getType() == types::TY_Nothing)<br>     Result = InputInfo(A, BaseInput);<br>-  else<br>+  else {<br>+    // We only have to generate a prefix for the host if this is not a top-level<br>+    // action.<br>+    std::string OffloadingPrefix = Action::GetOffloadingFileNamePrefix(<br>+        A->getOffloadingDeviceKind(), TC->getTriple().normalize(),<br>+        /*CreatePrefixForHost=*/!!A->getOffloadingHostActiveKinds() &&<br>+            !AtTopLevel);<br>     Result = InputInfo(A, GetNamedOutputPath(C, *JA, BaseInput, BoundArch,<br>                                              AtTopLevel, MultipleArchs,<br>-                                             TC->getTriple().normalize()),<br>+                                             OffloadingPrefix),<br>                        BaseInput);<br>+  }<br><br>   if (CCCPrintBindings && !CCGenDiagnostics) {<br>     llvm::errs() << "# \"" << T->getToolChain().getTripleString() << '"'<br>@@ -3130,12 +3182,28 @@ InputInfo Driver::BuildJobsForActionNoCa<br>       if (i + 1 != e)<br>         llvm::errs() << ", ";<br>     }<br>-    llvm::errs() << "], output: " << Result.getAsString() << "\n";<br>+    if (UnbundlingResults.empty())<br>+      llvm::errs() << "], output: " << Result.getAsString() << "\n";<br>+    else {<br>+      llvm::errs() << "], outputs: [";<br>+      for (unsigned i = 0, e = UnbundlingResults.size(); i != e; ++i) {<br>+        llvm::errs() << UnbundlingResults[i].getAsString();<br>+        if (i + 1 != e)<br>+          llvm::errs() << ", ";<br>+      }<br>+      llvm::errs() << "] \n";<br>+    }<br>   } else {<br>-    T->ConstructJob(<br>-        C, *JA, Result, InputInfos,<br>-        C.getArgsForToolChain(TC, BoundArch, JA->getOffloadingDeviceKind()),<br>-        LinkingOutput);<br>+    if (UnbundlingResults.empty())<br>+      T->ConstructJob(<br>+          C, *JA, Result, InputInfos,<br>+          C.getArgsForToolChain(TC, BoundArch, JA->getOffloadingDeviceKind()),<br>+          LinkingOutput);<br>+    else<br>+      T->ConstructJob(<br>+          C, *JA, UnbundlingResults, InputInfos,<br>+          C.getArgsForToolChain(TC, BoundArch, JA->getOffloadingDeviceKind()),<br>+          LinkingOutput);<br>   }<br>   return Result;<br> }<br>@@ -3182,7 +3250,7 @@ const char *Driver::GetNamedOutputPath(C<br>                                        const char *BaseInput,<br>                                        StringRef BoundArch, bool AtTopLevel,<br>                                        bool MultipleArchs,<br>-                                       StringRef NormalizedTriple) const {<br>+                                       StringRef OffloadingPrefix) const {<br>   llvm::PrettyStackTraceString CrashInfo("Computing output path");<br>   // Output to a user requested destination?<br>   if (AtTopLevel && !isa<DsymutilJobAction>(JA) && !isa<VerifyJobAction>(JA)) {<br>@@ -3268,7 +3336,7 @@ const char *Driver::GetNamedOutputPath(C<br>           MakeCLOutputFilename(C.getArgs(), "", BaseName, types::TY_Image);<br>     } else {<br>       SmallString<128> Output(getDefaultImageName());<br>-      Output += JA.getOffloadingFileNamePrefix(NormalizedTriple);<br>+      Output += OffloadingPrefix;<br>       if (MultipleArchs && !BoundArch.empty()) {<br>         Output += "-";<br>         Output.append(BoundArch);<br>@@ -3285,7 +3353,7 @@ const char *Driver::GetNamedOutputPath(C<br>     if (!types::appendSuffixForType(JA.getType()))<br>       End = BaseName.rfind('.');<br>     SmallString<128> Suffixed(BaseName.substr(0, End));<br>-    Suffixed += JA.getOffloadingFileNamePrefix(NormalizedTriple);<br>+    Suffixed += OffloadingPrefix;<br>     if (MultipleArchs && !BoundArch.empty()) {<br>       Suffixed += "-";<br>       Suffixed.append(BoundArch);<br><br>Modified: cfe/trunk/lib/Driver/Tool.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/lib/Driver/Tool.cpp?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/lib/Driver/Tool.cpp (original)<br>+++ cfe/trunk/lib/Driver/Tool.cpp Thu Oct 27 13:14:55 2016<br>@@ -8,6 +8,7 @@<br> //===----------------------------------------------------------------------===//<br><br> #include "clang/Driver/Tool.h"<br>+#include "InputInfo.h"<br><br> using namespace clang::driver;<br><br>@@ -21,3 +22,12 @@ Tool::Tool(const char *_Name, const char<br><br> Tool::~Tool() {<br> }<br>+<br>+void Tool::ConstructJob(Compilation &C, const JobAction &JA,<br>+                        const InputInfoList &Outputs,<br>+                        const InputInfoList &Inputs,<br>+                        const llvm::opt::ArgList &TCArgs,<br>+                        const char *LinkingOutput) const {<br>+  assert(Outputs.size() == 1 && "Expected only one output by default!");<br>+  ConstructJob(C, JA, Outputs.front(), Inputs, TCArgs, LinkingOutput);<br>+};<br><br>Modified: cfe/trunk/lib/Driver/Tools.cpp<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/lib/Driver/Tools.cpp?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/lib/Driver/Tools.cpp (original)<br>+++ cfe/trunk/lib/Driver/Tools.cpp Thu Oct 27 13:14:55 2016<br>@@ -6222,7 +6222,8 @@ void Clang::ConstructJob(Compilation &C,<br>         if (!JA.isDeviceOffloading(Action::OFK_None) &&<br>             !JA.isDeviceOffloading(Action::OFK_Host)) {<br>           llvm::sys::path::replace_extension(F, "");<br>-          F += JA.getOffloadingFileNamePrefix(Triple.normalize());<br>+          F += Action::GetOffloadingFileNamePrefix(JA.getOffloadingDeviceKind(),<br>+                                                   Triple.normalize());<br>           F += "-";<br>           F += JA.getOffloadingArch();<br>         }<br>@@ -7058,6 +7059,7 @@ void OffloadBundler::ConstructJob(Compil<br>                                   const InputInfoList &Inputs,<br>                                   const llvm::opt::ArgList &TCArgs,<br>                                   const char *LinkingOutput) const {<br>+  // The version with only one output is expected to refer to a bundling job.<br>   assert(isa<OffloadBundlingJobAction>(JA) && "Expecting bundling job!");<br><br>   // The bundling command looks like this:<br>@@ -7114,6 +7116,68 @@ void OffloadBundler::ConstructJob(Compil<br><br>   // All the inputs are encoded as commands.<br>   C.addCommand(llvm::make_unique<Command>(<br>+      JA, *this,<br>+      TCArgs.MakeArgString(getToolChain().GetProgramPath(getShortName())),<br>+      CmdArgs, None));<br>+}<br>+<br>+void OffloadBundler::ConstructJob(Compilation &C, const JobAction &JA,<br>+                                  const InputInfoList &Outputs,<br>+                                  const InputInfoList &Inputs,<br>+                                  const llvm::opt::ArgList &TCArgs,<br>+                                  const char *LinkingOutput) const {<br>+  // The version with multiple outputs is expected to refer to a unbundling job.<br>+  auto &UA = cast<OffloadUnbundlingJobAction>(JA);<br>+<br>+  // The unbundling command looks like this:<br>+  // clang-offload-bundler -type=bc<br>+  //   -targets=host-triple,openmp-triple1,openmp-triple2<br>+  //   -inputs=input_file<br>+  //   -outputs=unbundle_file_host,unbundle_file_tgt1,unbundle_file_tgt2"<br>+  //   -unbundle<br>+<br>+  ArgStringList CmdArgs;<br>+<br>+  assert(Inputs.size() == 1 && "Expecting to unbundle a single file!");<br>+  InputInfo Input = Inputs.front();<br>+<br>+  // Get the type.<br>+  CmdArgs.push_back(TCArgs.MakeArgString(<br>+      Twine("-type=") + types::getTypeTempSuffix(Input.getType())));<br>+<br>+  // Get the targets.<br>+  SmallString<128> Triples;<br>+  Triples += "-targets=";<br>+  auto DepInfo = UA.getDependentActionsInfo();<br>+  for (unsigned I = 0; I < DepInfo.size(); ++I) {<br>+    if (I)<br>+      Triples += ',';<br>+<br>+    auto &Dep = DepInfo[I];<br>+    Triples += Action::GetOffloadKindName(Dep.DependentOffloadKind);<br>+    Triples += '-';<br>+    Triples += Dep.DependentToolChain->getTriple().normalize();<br>+  }<br>+<br>+  CmdArgs.push_back(TCArgs.MakeArgString(Triples));<br>+<br>+  // Get bundled file command.<br>+  CmdArgs.push_back(<br>+      TCArgs.MakeArgString(Twine("-inputs=") + Input.getFilename()));<br>+<br>+  // Get unbundled files command.<br>+  SmallString<128> UB;<br>+  UB += "-outputs=";<br>+  for (unsigned I = 0; I < Outputs.size(); ++I) {<br>+    if (I)<br>+      UB += ',';<br>+    UB += Outputs[I].getFilename();<br>+  }<br>+  CmdArgs.push_back(TCArgs.MakeArgString(UB));<br>+  CmdArgs.push_back("-unbundle");<br>+<br>+  // All the inputs are encoded as commands.<br>+  C.addCommand(llvm::make_unique<Command>(<br>       JA, *this,<br>       TCArgs.MakeArgString(getToolChain().GetProgramPath(getShortName())),<br>       CmdArgs, None));<br><br>Modified: cfe/trunk/lib/Driver/Tools.h<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/lib/Driver/Tools.h?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/lib/Driver/Tools.h (original)<br>+++ cfe/trunk/lib/Driver/Tools.h Thu Oct 27 13:14:55 2016<br>@@ -148,6 +148,10 @@ public:<br>                     const InputInfo &Output, const InputInfoList &Inputs,<br>                     const llvm::opt::ArgList &TCArgs,<br>                     const char *LinkingOutput) const override;<br>+  void ConstructJob(Compilation &C, const JobAction &JA,<br>+                    const InputInfoList &Outputs, const InputInfoList &Inputs,<br>+                    const llvm::opt::ArgList &TCArgs,<br>+                    const char *LinkingOutput) const override;<br> };<br><br> /// \brief Base class for all GNU tools that provide the same behavior when<br><br>Modified: cfe/trunk/test/Driver/<a href="http://cuda-bindings.cu" target="_blank" >cuda-bindings.cu</a><br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/test/Driver/<a href="http://cuda-bindings.cu?rev=285326&" target="_blank" >cuda-bindings.cu?rev=285326&</a>r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/test/Driver/<a href="http://cuda-bindings.cu" target="_blank" >cuda-bindings.cu</a> (original)<br>+++ cfe/trunk/test/Driver/<a href="http://cuda-bindings.cu" target="_blank" >cuda-bindings.cu</a> Thu Oct 27 13:14:55 2016<br>@@ -34,7 +34,7 @@<br> //<br> // RUN: %clang -target powerpc64le-ibm-linux-gnu -ccc-print-bindings --cuda-gpu-arch=sm_30 %s -S 2>&1 \<br> // RUN: | FileCheck -check-prefix=ASM %s<br>-// ASM-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>+// ASM-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.s"<br> // ASM-DAG: # "powerpc64le-ibm-linux-gnu" - "clang",{{.*}} output: "cuda-bindings.s"<br><br> //<br>@@ -62,8 +62,8 @@<br> // RUN: %clang -target powerpc64le-ibm-linux-gnu -ccc-print-bindings \<br> // RUN:        --cuda-gpu-arch=sm_30 --cuda-gpu-arch=sm_35 %s -S 2>&1 \<br> // RUN: | FileCheck -check-prefix=ASM2 %s<br>-// ASM2-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>-// ASM2-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_35.s"<br>+// ASM2-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>+// ASM2-DAG: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_35.s"<br> // ASM2-DAG: # "powerpc64le-ibm-linux-gnu" - "clang",{{.*}} output: "cuda-bindings.s"<br><br> //<br>@@ -101,7 +101,7 @@<br> // RUN: | FileCheck -check-prefix=DBIN %s<br> // DBIN: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output:<br> // DBIN-NOT: cuda-bindings-device-cuda-nvptx64<br>-// DBIN: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.o"<br>+// DBIN: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.o"<br><br> //<br> // Test single gpu architecture up to the assemble phase in device-only<br>@@ -110,7 +110,7 @@<br> // RUN: %clang -target powerpc64le-ibm-linux-gnu -ccc-print-bindings \<br> // RUN:        --cuda-gpu-arch=sm_30 %s --cuda-device-only -S 2>&1 \<br> // RUN: | FileCheck -check-prefix=DASM %s<br>-// DASM: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>+// DASM: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.s"<br><br> //<br> // Test two gpu architectures with complete compilation in device-only<br>@@ -121,10 +121,10 @@<br> // RUN: | FileCheck -check-prefix=DBIN2 %s<br> // DBIN2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output:<br> // DBIN2-NOT: cuda-bindings-device-cuda-nvptx64<br>-// DBIN2: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.o"<br>+// DBIN2: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.o"<br> // DBIN2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output:<br> // DBIN2-NOT: cuda-bindings-device-cuda-nvptx64<br>-// DBIN2: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_35.o"<br>+// DBIN2: # "nvptx64-nvidia-cuda" - "NVPTX::Assembler",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_35.o"<br><br> //<br> // Test two gpu architectures up to the assemble phase in device-only<br>@@ -133,5 +133,5 @@<br> // RUN: %clang -target powerpc64le-ibm-linux-gnu -ccc-print-bindings \<br> // RUN:        --cuda-gpu-arch=sm_30 --cuda-gpu-arch=sm_35 %s --cuda-device-only -S 2>&1 \<br> // RUN: | FileCheck -check-prefix=DASM2 %s<br>-// DASM2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>-// DASM2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-device-cuda-nvptx64-nvidia-cuda-sm_35.s"<br>+// DASM2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_30.s"<br>+// DASM2: # "nvptx64-nvidia-cuda" - "clang",{{.*}} output: "cuda-bindings-cuda-nvptx64-nvidia-cuda-sm_35.s"<br><br>Modified: cfe/trunk/test/Driver/openmp-offload.c<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/test/Driver/openmp-offload.c?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/test/Driver/openmp-offload.c (original)<br>+++ cfe/trunk/test/Driver/openmp-offload.c Thu Oct 27 13:14:55 2016<br>@@ -210,8 +210,8 @@<br> // CHK-LKS: TARGET(binary)<br> // CHK-LKS-REG: INPUT([[T1BIN:.+\.out]])<br> // CHK-LKS-REG: INPUT([[T2BIN:.+\.out]])<br>-// CHK-LKS-ST: INPUT([[T1BIN:.+\.out-device-openmp-powerpc64le-ibm-linux-gnu]])<br>-// CHK-LKS-ST: INPUT([[T2BIN:.+\.out-device-openmp-x86_64-pc-linux-gnu]])<br>+// CHK-LKS-ST: INPUT([[T1BIN:.+\.out-openmp-powerpc64le-ibm-linux-gnu]])<br>+// CHK-LKS-ST: INPUT([[T2BIN:.+\.out-openmp-x86_64-pc-linux-gnu]])<br> // CHK-LKS: SECTIONS<br> // CHK-LKS: {<br> // CHK-LKS:   .omp_offloading :<br>@@ -389,3 +389,92 @@<br> // CHK-BUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTASM:.+\.s]]" "-x" "ir" "[[HOSTBC]]"<br> // CHK-BUJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "powerpc64le--linux" "-filetype" "obj" {{.*}}"-o" "[[HOSTOBJ:.+\.o]]" "[[HOSTASM]]"<br> // CHK-BUJOBS-ST: clang-offload-bundler" "-type=o" "-targets=openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu,host-powerpc64le--linux" "-outputs=[[RES:.+\.o]]" "-inputs=[[T1OBJ]],[[T2OBJ]],[[HOSTOBJ]]"<br>+<br>+/// ###########################################################################<br>+<br>+/// Check separate compilation with offloading - unbundling jobs construct<br>+// RUN:   touch %t.i<br>+// RUN:   %clang -###  -fopenmp -o %t.out -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.i 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBJOBS %s<br>+// RUN:   %clang -### -fopenmp -o %t.out -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.i -save-temps 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBJOBS-ST %s<br>+// RUN:   touch %t.o<br>+// RUN:   %clang -###  -fopenmp -o %t.out -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.o 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBJOBS2 %s<br>+// RUN:   %clang -### -fopenmp -o %t.out -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.o -save-temps 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBJOBS2-ST %s<br>+<br>+// Unbundle and create host BC.<br>+// CHK-UBJOBS: clang-offload-bundler" "-type=i" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.i]]" "-outputs=[[HOSTPP:.+\.i]],[[T1PP:.+\.i]],[[T2PP:.+\.i]]" "-unbundle"<br>+// CHK-UBJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-llvm-bc"  {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTBC:.+\.bc]]" "-x" "cpp-output" "[[HOSTPP]]" "-fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu"<br>+// CHK-UBJOBS-ST: clang-offload-bundler" "-type=i" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.i]]" "-outputs=[[HOSTPP:.+\.i]],[[T1PP:.+\.i]],[[T2PP:.+\.i]]" "-unbundle"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-llvm-bc"  {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTBC:.+\.bc]]" "-x" "cpp-output" "[[HOSTPP]]" "-fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu"<br>+<br>+// Create target 1 object.<br>+// CHK-UBJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1OBJ:.+\.o]]" "-x" "cpp-output" "[[T1PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBJOBS: ld" {{.*}}"-o" "[[T1BIN:.+\.out]]" {{.*}}"[[T1OBJ]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-emit-llvm-bc" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1BC:.+\.bc]]" "-x" "cpp-output" "[[T1PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1ASM:.+\.s]]" "-x" "ir" "[[T1BC]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "powerpc64le-ibm-linux-gnu" "-filetype" "obj" {{.*}}"-o" "[[T1OBJ:.+\.o]]" "[[T1ASM]]"<br>+// CHK-UBJOBS-ST: ld" {{.*}}"-o" "[[T1BIN:.+\.out-openmp-powerpc64le-ibm-linux-gnu]]" {{.*}}"[[T1OBJ]]"<br>+<br>+// Create target 2 object.<br>+// CHK-UBJOBS: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2OBJ:.+\.o]]" "-x" "cpp-output" "[[T2PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBJOBS: ld" {{.*}}"-o" "[[T2BIN:.+\.out]]" {{.*}}"[[T2OBJ]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-emit-llvm-bc" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2BC:.+\.bc]]" "-x" "cpp-output" "[[T2PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2ASM:.+\.s]]" "-x" "ir" "[[T2BC]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "x86_64-pc-linux-gnu" "-filetype" "obj" {{.*}}"-o" "[[T2OBJ:.+\.o]]" "[[T2ASM]]"<br>+// CHK-UBJOBS-ST: ld" {{.*}}"-o" "[[T2BIN:.+\.out-openmp-x86_64-pc-linux-gnu]]" {{.*}}"[[T2OBJ]]"<br>+<br>+// Create binary.<br>+// CHK-UBJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTOBJ:.+\.o]]" "-x" "ir" "[[HOSTBC]]"<br>+// CHK-UBJOBS: ld" {{.*}}"-o" "[[HOSTBIN:.+\.out]]" {{.*}}"[[HOSTOBJ]]" {{.*}}"-T" "[[LKS:.+\.lk]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTASM:.+\.s]]" "-x" "ir" "[[HOSTBC]]"<br>+// CHK-UBJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "powerpc64le--linux" "-filetype" "obj" {{.*}}"-o" "[[HOSTOBJ:.+\.o]]" "[[HOSTASM]]"<br>+// CHK-UBJOBS-ST: ld" {{.*}}"-o" "[[HOSTBIN:.+\.out]]" {{.*}}"[[HOSTOBJ]]" {{.*}}"-T" "[[LKS:.+\.lk]]"<br>+<br>+// Unbundle object file.<br>+// CHK-UBJOBS2: clang-offload-bundler" "-type=o" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.o]]" "-outputs=[[HOSTOBJ:.+\.o]],[[T1OBJ:.+\.o]],[[T2OBJ:.+\.o]]" "-unbundle"<br>+// CHK-UBJOBS2: ld" {{.*}}"-o" "[[T1BIN:.+\.out]]" {{.*}}"[[T1OBJ]]"<br>+// CHK-UBJOBS2: ld" {{.*}}"-o" "[[T2BIN:.+\.out]]" {{.*}}"[[T2OBJ]]"<br>+// CHK-UBJOBS2: ld" {{.*}}"-o" "[[HOSTBIN:.+\.out]]" {{.*}}"[[HOSTOBJ]]" {{.*}}"-T" "[[LKS:.+\.lk]]"<br>+// CHK-UBJOBS2-ST: clang-offload-bundler" "-type=o" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.o]]" "-outputs=[[HOSTOBJ:.+\.o]],[[T1OBJ:.+\.o]],[[T2OBJ:.+\.o]]" "-unbundle"<br>+// CHK-UBJOBS2-ST: ld" {{.*}}"-o" "[[T1BIN:.+\.out-openmp-powerpc64le-ibm-linux-gnu]]" {{.*}}"[[T1OBJ]]"<br>+// CHK-UBJOBS2-ST: ld" {{.*}}"-o" "[[T2BIN:.+\.out-openmp-x86_64-pc-linux-gnu]]" {{.*}}"[[T2OBJ]]"<br>+// CHK-UBJOBS2-ST: ld" {{.*}}"-o" "[[HOSTBIN:.+\.out]]" {{.*}}"[[HOSTOBJ]]" {{.*}}"-T" "[[LKS:.+\.lk]]"<br>+<br>+/// ###########################################################################<br>+<br>+/// Check separate compilation with offloading - unbundling/bundling jobs<br>+/// construct<br>+// RUN:   touch %t.i<br>+// RUN:   %clang -### -fopenmp -c %t.o -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.i 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBUJOBS %s<br>+// RUN:   %clang -### -fopenmp -c %t.o -lsomelib -target powerpc64le-linux -fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu %t.i -save-temps 2>&1 \<br>+// RUN:   | FileCheck -check-prefix=CHK-UBUJOBS-ST %s<br>+<br>+// Unbundle and create host BC.<br>+// CHK-UBUJOBS: clang-offload-bundler" "-type=i" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.i]]" "-outputs=[[HOSTPP:.+\.i]],[[T1PP:.+\.i]],[[T2PP:.+\.i]]" "-unbundle"<br>+// CHK-UBUJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-llvm-bc"  {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTBC:.+\.bc]]" "-x" "cpp-output" "[[HOSTPP]]" "-fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu"<br>+<br>+// CHK-UBUJOBS-ST: clang-offload-bundler" "-type=i" "-targets=host-powerpc64le--linux,openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu" "-inputs=[[INPUT:.+\.i]]" "-outputs=[[HOSTPP:.+\.i]],[[T1PP:.+\.i]],[[T2PP:.+\.i]]" "-unbundle"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-llvm-bc"  {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTBC:.+\.bc]]" "-x" "cpp-output" "[[HOSTPP]]" "-fopenmp-targets=powerpc64le-ibm-linux-gnu,x86_64-pc-linux-gnu"<br>+<br>+// Create target 1 object.<br>+// CHK-UBUJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1OBJ:.+\.o]]" "-x" "cpp-output" "[[T1PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-emit-llvm-bc" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1BC:.+\.bc]]" "-x" "cpp-output" "[[T1PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le-ibm-linux-gnu" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[T1ASM:.+\.s]]" "-x" "ir" "[[T1BC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "powerpc64le-ibm-linux-gnu" "-filetype" "obj" {{.*}}"-o" "[[T1OBJ:.+\.o]]" "[[T1ASM]]"<br>+<br>+// Create target 2 object.<br>+// CHK-UBUJOBS: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2OBJ:.+\.o]]" "-x" "cpp-output" "[[T2PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-emit-llvm-bc" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2BC:.+\.bc]]" "-x" "cpp-output" "[[T2PP]]" "-fopenmp-is-device" "-fopenmp-host-ir-file-path" "[[HOSTBC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "x86_64-pc-linux-gnu" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[T2ASM:.+\.s]]" "-x" "ir" "[[T2BC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "x86_64-pc-linux-gnu" "-filetype" "obj" {{.*}}"-o" "[[T2OBJ:.+\.o]]" "[[T2ASM]]"<br>+<br>+// Create binary.<br>+// CHK-UBUJOBS: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-emit-obj" {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTOBJ:.+\.o]]" "-x" "ir" "[[HOSTBC]]"<br>+// CHK-UBUJOBS: clang-offload-bundler" "-type=o" "-targets=openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu,host-powerpc64le--linux" "-outputs=[[RES:.+\.o]]" "-inputs=[[T1OBJ]],[[T2OBJ]],[[HOSTOBJ]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1" "-triple" "powerpc64le--linux" "-S" {{.*}}"-fopenmp" {{.*}}"-o" "[[HOSTASM:.+\.s]]" "-x" "ir" "[[HOSTBC]]"<br>+// CHK-UBUJOBS-ST: clang{{.*}}" "-cc1as" "-triple" "powerpc64le--linux" "-filetype" "obj" {{.*}}"-o" "[[HOSTOBJ:.+\.o]]" "[[HOSTASM]]"<br>+// CHK-UBUJOBS-ST: clang-offload-bundler" "-type=o" "-targets=openmp-powerpc64le-ibm-linux-gnu,openmp-x86_64-pc-linux-gnu,host-powerpc64le--linux" "-outputs=[[RES:.+\.o]]" "-inputs=[[T1OBJ]],[[T2OBJ]],[[HOSTOBJ]]"<br><br>Modified: cfe/trunk/test/Driver/opt-record.c<br>URL: <a href="http://llvm.org/viewvc/llvm" target="_blank" >http://llvm.org/viewvc/llvm</a>-project/cfe/trunk/test/Driver/opt-record.c?rev=285326&r1=285325&r2=285326&view=diff<br>==============================================================================<br>--- cfe/trunk/test/Driver/opt-record.c (original)<br>+++ cfe/trunk/test/Driver/opt-record.c Thu Oct 27 13:14:55 2016<br>@@ -11,7 +11,7 @@<br><br> // CHECK-NO-O: "-cc1"<br> // CHECK-NO-O-DAG: "-opt-record-file" "opt-record.opt.yaml"<br>-// CHECK-CUDA-DEV-DAG: "-opt-record-file" "opt-record-device-cuda-{{nvptx64|nvptx}}-nvidia-cuda-sm_20.opt.yaml"<br>+// CHECK-CUDA-DEV-DAG: "-opt-record-file" "opt-record-cuda-{{nvptx64|nvptx}}-nvidia-cuda-sm_20.opt.yaml"<br><br> // CHECK-EQ: "-cc1"<br> // CHECK-EQ: "-opt-record-file" "BAR.txt"<br><br><br>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@lists.llvm.org" target="_blank" >cfe-commits@lists.llvm.org</a><br><a href="http://lists.llvm.org/cgi-bin" target="_blank" >http://lists.llvm.org/cgi-bin</a>/mailman/listinfo/cfe-commits<o:p></o:p></p></div></div></blockquote>
<div><p><span style="font-size:10.5pt;font-family:"Helvetica Neue"" > </span><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" ><o:p></o:p></span></p></div></div></div></div>
<p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div></div></div></div>
<div id="MIMEAttachInfoDiv" ><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div>
<div id="MIMEAttachInfoDiv" ><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div></blockquote>
<div><p><span style="font-size:10.5pt;font-family:"Arial","sans-serif"" > <o:p></o:p></span></p></div></div>
<p><o:p> </o:p></p></div></div></div>
<div id="MIMEAttachInfoDiv" style="display:none" title="octet-stream|smime.p7s" > </div></blockquote>
<div dir="ltr" > </div></div><BR>