[Openmp-commits] [openmp] r242298 - Large Refactor of CMake build system

[Peyton, Jonathan L]

Thanks Hans!

-- Johnny

-----Original Message-----
From: hwennborg at google.com [mailto:hwennborg at google.com] On Behalf Of Hans Wennborg
Sent: Wednesday, July 15, 2015 4:43 PM
To: Peyton, Jonathan L
Cc: openmp-commits at cs.uiuc.edu
Subject: Re: [Openmp-commits] [openmp] r242298 - Large Refactor of CMake build system

Hi Jonathan,

I think this is a reasonable request as we're still early in the process and this can be considered a fix towards completing an existing feature.

I've merged this and the follow-up commit in r242335 and r242336, respectively.

Thanks,
Hans

On Wed, Jul 15, 2015 at 2:29 PM, Peyton, Jonathan L <jonathan.l.peyton at intel.com> wrote:
> Hans,
>
> We were trying to get this into the 3.7.0 release so users could have this library be the default for the -fopenmp flag.  We missed the deadline by one day.  There were two commits that refactor the CMake code to be up to LLVM standards.  I would greatly appreciate if this (and the follow up commit r242301) were committed to the 3.7.0 openmp branch.  I know they are large, but they are key to getting the LLVM OpenMP Runtime library set as the default for clang.
>
> -- Johnny
>
> -----Original Message-----
> From: openmp-commits-bounces at cs.uiuc.edu 
> [mailto:openmp-commits-bounces at cs.uiuc.edu] On Behalf Of Jonathan 
> Peyton
> Sent: Wednesday, July 15, 2015 11:06 AM
> To: openmp-commits at cs.uiuc.edu
> Subject: [Openmp-commits] [openmp] r242298 - Large Refactor of CMake 
> build system
>
> Author: jlpeyton
> Date: Wed Jul 15 11:05:30 2015
> New Revision: 242298
>
> URL: http://llvm.org/viewvc/llvm-project?rev=242298&view=rev
> Log:
> Large Refactor of CMake build system