<div dir="ltr">Jonathan, that is not the code review policy of the LLVM community. Please, do not try to force code review by setting a time table. Once you ask for code review, it is your responsibility to wait for your reviewers to approve the patch.<br><div><br></div><div>I'm sorry that last week didn't get you a code review. Lots of folks were on vacation, and in the US it was a short week due to holidays. Unfortunately, this is the reality, and doesn't mean you should rush the patch.</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 7, 2015 at 8:53 AM Peyton, Jonathan L <<a href="mailto:jonathan.l.peyton@intel.com">jonathan.l.peyton@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I feel there has been ample opportunity to review the new iterations of this patch. And if no one objects, I'm going to commit this tomorrow. Post commit reviews are always welcome.<br>
<br>
-- Johnny<br>
<br>
-----Original Message-----<br>
From: Jack Howarth [mailto:<a href="mailto:howarth.mailing.lists@gmail.com" target="_blank">howarth.mailing.lists@gmail.com</a>]<br>
Sent: Tuesday, July 7, 2015 7:25 AM<br>
To: <a href="mailto:reviews%2BD10656%2Bpublic%2Bdda1fdcafb4c32bf@reviews.llvm.org" target="_blank">reviews+D10656+public+dda1fdcafb4c32bf@reviews.llvm.org</a><br>
Cc: Peyton, Jonathan L; Chandler Carruth; C Bergström; <a href="mailto:openmp-commits@dcs-maillist2.engr.illinois.edu" target="_blank">openmp-commits@dcs-maillist2.engr.illinois.edu</a>; Cownie, James H; <a href="mailto:howarth.mailing.lists@apple.com" target="_blank">howarth.mailing.lists@apple.com</a>; <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
Subject: Re: [PATCH] LLVM OpenMP CMake Overhaul<br>
<br>
Jonathan,<br>
We are now a week away from the branching of 3.7.0. Do you have an ETA for when you plan to commit this patch to trunk?<br>
Jack<br>
<br>
On Tue, Jun 30, 2015 at 5:08 PM, Jonathan Peyton <<a href="mailto:jonathan.l.peyton@intel.com" target="_blank">jonathan.l.peyton@intel.com</a>> wrote:<br>
> Addressing most of Chandler's remaining issues with this third iteration.<br>
><br>
> - There is no longer a LibompSourceFiles.cmake file. The files are<br>
> grabbed inside src/CMakeLists.txt<br>
> - The -m32 flag is guarded by if(CMAKE_SIZEOF_VOID_P EQUAL 8) so we only use it when using a x86_64 toolchain to compile i386.<br>
> - Removed passing in of _KMP_BUILD_TIME.<br>
> - I've gotten the library to build using Unix Makefiles, NMake Makefiles, and NInja on Windows in standalone mode. There is still one issue when trying to build in a Windows LLVM tree which is that LLVM adds numerous -wXYZ flags by using add_llvm_definitions() which causes the resource compiler (RC.exe) to include them in its command which causes an unknown flag error.<br>
><br>
><br>
> REPOSITORY<br>
> rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D10656" rel="noreferrer" target="_blank">http://reviews.llvm.org/D10656</a><br>
><br>
> Files:<br>
> runtime/CMakeLists.txt<br>
> runtime/cmake/BuildPLRules.cmake<br>
> runtime/cmake/Clang/AsmFlags.cmake<br>
> runtime/cmake/Clang/CFlags.cmake<br>
> runtime/cmake/CommonFlags.cmake<br>
> runtime/cmake/Definitions.cmake<br>
> runtime/cmake/GNU/AsmFlags.cmake<br>
> runtime/cmake/GNU/CFlags.cmake<br>
> runtime/cmake/GNU/FortranFlags.cmake<br>
> runtime/cmake/GetArchitecture.cmake<br>
> runtime/cmake/HelperFunctions.cmake<br>
> runtime/cmake/Intel/AsmFlags.cmake<br>
> runtime/cmake/Intel/CFlags.cmake<br>
> runtime/cmake/Intel/FortranFlags.cmake<br>
> runtime/cmake/LibompCheckFortranFlag.cmake<br>
> runtime/cmake/LibompCheckLinkerFlag.cmake<br>
> runtime/cmake/LibompDefinitions.cmake<br>
> runtime/cmake/LibompExports.cmake<br>
> runtime/cmake/LibompGetArchitecture.cmake<br>
> runtime/cmake/LibompHandleFlags.cmake<br>
> runtime/cmake/LibompMicroTests.cmake<br>
> runtime/cmake/LibompUtils.cmake<br>
> runtime/cmake/MSVC/AsmFlags.cmake<br>
> runtime/cmake/MSVC/CFlags.cmake<br>
> runtime/cmake/MicroTests.cmake<br>
> runtime/cmake/PerlFlags.cmake<br>
> runtime/cmake/SourceFiles.cmake<br>
> runtime/cmake/config-ix.cmake<br>
> runtime/src/CMakeLists.txt<br>
><br>
> EMAIL PREFERENCES<br>
> <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" rel="noreferrer" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<br>
_______________________________________________<br>
Openmp-commits mailing list<br>
<a href="mailto:Openmp-commits@dcs-maillist2.engr.illinois.edu" target="_blank">Openmp-commits@dcs-maillist2.engr.illinois.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/openmp-commits" rel="noreferrer" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/openmp-commits</a><br>
</blockquote></div>