<div dir="ltr">Thanks for clarifying. I've prepared a patch for exclude it from the cmake build on Windows: <a href="https://reviews.llvm.org/D29801">https://reviews.llvm.org/D29801</a> Please take a look.<div><br></div><div>Thanks,</div><div>Hans<br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 9, 2017 at 4:55 PM, Georgios Rokos <span dir="ltr"><<a href="mailto:grokos@us.ibm.com" target="_blank">grokos@us.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><p>Hans,<br><br>We have not tested libomptarget on Windows and indeed we don't have any machine to test the library on.<br><br>Anyway, libomptarget in its current version comes with two plugins (elf64, CUDA), both of which expect the user application fat binary to be in ELF format, so by definition those plugins are not useful on Windows and consequently the whole libomptarget library cannot be used right now.<br><br>George<br><br><img width="16" height="16" src="cid:1__=0ABB0A50DF97B29E8f9e8a93df938690918c0AB@" border="0" alt="Inactive hide details for Hans Wennborg ---09/02/2017 17:21:15---It's difficult for me to try and debug this as the snapshots a"><font color="#424282">Hans Wennborg ---09/02/2017 17:21:15---It's difficult for me to try and debug this as the snapshots are built by a lengthy batch process.</font><br><br><font size="2" color="#5F5F5F">From:        </font><font size="2">Hans Wennborg <<a href="mailto:hans@chromium.org" target="_blank">hans@chromium.org</a>></font><br><font size="2" color="#5F5F5F">To:        </font><font size="2">"Hahnfeld, Jonas" <<a href="mailto:Hahnfeld@itc.rwth-aachen.de" target="_blank">Hahnfeld@itc.rwth-aachen.de</a>></font><br><font size="2" color="#5F5F5F">Cc:        </font><font size="2">Georgios Rokos/Watson/IBM@IBMUS, "openmp-commits (<a href="mailto:openmp-commits@lists.llvm.org" target="_blank">openmp-commits@lists.llvm.org</a><wbr>)" <<a href="mailto:openmp-commits@lists.llvm.org" target="_blank">openmp-commits@lists.llvm.org</a><wbr>></font><br><font size="2" color="#5F5F5F">Date:        </font><font size="2">09/02/2017 17:21</font><span class="gmail-"><br><font size="2" color="#5F5F5F">Subject:        </font><font size="2">Re: [Openmp-commits] [openmp] r293094 - [OpenMP] Initial implementation of OpenMP offloading library - libomptarget.</font><br></span><font size="2" color="#5F5F5F">Sent by:        </font><font size="2"><a href="mailto:hwennborg@google.com" target="_blank">hwennborg@google.com</a></font><br></p><hr width="100%" size="2" align="left" noshade style="color:rgb(128,145,165)"><div><div class="gmail-h5"><br><br><br><tt>It's difficult for me to try and debug this as the snapshots are built<br>by a lengthy batch process.<br><br>I've just removed the OpenMP runtime from the snapshots for now.<br><br>If you don't have a Windows machine setup for building, it sounds like<br>this isn't supported on Windows and should be disabled.<br><br>Thanks,<br>Hans<br><br>On Tue, Jan 31, 2017 at 12:02 AM, Hahnfeld, Jonas<br><<a href="mailto:Hahnfeld@itc.rwth-aachen.de" target="_blank">Hahnfeld@itc.rwth-aachen.de</a>> wrote:<br>> Hi Hans,<br>><br>>> -- LIBOMPTARGET: Building offloading runtime library libomptarget.<br>> should only be printed after -std=c++11 is checked, so libomptarget is<br>> probably built.<br>><br>>> CMake Error at projects/openmp/libomptarget/<wbr>CMakeLists.txt:101 (install):<br>>>   install Library TARGETS given no DESTINATION!<br>> is really weird because line 101 actually has a DESTINATION. However, this is<br>> only applied for shared libraries if I understand the CMake documentation for<br>> install() correctly.<br>> libomp specifies RUNTIME for WIN32 here... Would you mind testing with RUNTIME<br>> instead of LIRBARY or without any keyword at all? I don't currently have a<br>> Windows machine setup for building...<br>><br>> Regards,<br>> Jonas<br>><br>>> -----Original Message-----<br>>> From: <a href="mailto:hwennborg@google.com" target="_blank">hwennborg@google.com</a> [</tt><tt><a href="mailto:hwennborg@google.com" target="_blank">mailto:hwennborg@google.com</a></tt><tt>] On Behalf<br>>> Of Hans Wennborg<br>>> Sent: Monday, January 30, 2017 11:11 PM<br>>> To: George Rokos; Hahnfeld, Jonas<br>>> Cc: openmp-commits (<a href="mailto:openmp-commits@lists.llvm.org" target="_blank">openmp-commits@lists.llvm.org</a><wbr>)<br>>> Subject: Re: [Openmp-commits] [openmp] r293094 - [OpenMP] Initial<br>>> implementation of OpenMP offloading library - libomptarget.<br>>><br>>> Hi George,<br>>><br>>> On Wed, Jan 25, 2017 at 1:27 PM, George Rokos via Openmp-commits<br>>> <<a href="mailto:openmp-commits@lists.llvm.org" target="_blank">openmp-commits@lists.llvm.org</a><wbr>> wrote:<br>>> > Author: grokos<br>>> > Date: Wed Jan 25 15:27:24 2017<br>>> > New Revision: 293094<br>>> ><br>>> > URL: </tt><tt><a href="http://llvm.org/viewvc/llvm-project?rev=293094&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=293094&view=rev</a></tt><tt><br>>> > Log:<br>>> > [OpenMP] Initial implementation of OpenMP offloading library -<br>>> libomptarget.<br>>> ><br>>> > This is the patch upstreaming the device-agnostic part of libomptarget.<br>>> ><br>>> > Differential Revision: </tt><tt><a href="https://reviews.llvm.org/D14031" target="_blank">https://reviews.llvm.org/<wbr>D14031</a></tt><tt><br>>><br>>> I believe this broke self-hosting of clang+openmp on Windows.<br>>><br>>> I get the following CMake error:<br>>><br>>> -- LIBOMPTARGET: Building offloading runtime library libomptarget.<br>>> CMake Error at projects/openmp/libomptarget/<wbr>CMakeLists.txt:101 (install):<br>>>   install Library TARGETS given no DESTINATION!<br>>><br>>> I'm not sure that makes any sense.<br>>><br>>> It seems the library does not get built with MSVC, probably because it<br>>> doesn't support the -std=c++11 option. But clang-cl does.<br>>><br>>> Is this library expected to build on Windows? Otherwise, can we disable it?<br>>><br>>> For this week's Windows snapshot, I'll just exclude OpenMP.<br>>><br>>> Thanks,<br>>> Hans<br><br></tt><br><br><br>
</div></div><p></p></div>
</blockquote></div><br></div></div></div>