[llvm-commits] [PATCH] Fix incorrect llvm-config output by adding support for OptionalLibrary to llvm-build

[Malea, Daniel]

Ping.

From: Malea, Daniel
Sent: Monday, April 09, 2012 1:23 PM
To: llvm-commits at cs.uiuc.edu
Cc: Alberto Magni (alberto.magni86 at gmail.com)
Subject: [PATCH] Fix incorrect llvm-config output by adding support for OptionalLibrary to llvm-build

Hi all,

Please find the attached patch that addresses concerns identified by Alberto Magni in the llvm-dev thread:
http://lists.cs.uiuc.edu/pipermail/llvmdev/2012-March/048521.html

The patch adds a new "type" of library (in llvm-build terms) named OptionalLibrary which must be explicitly enabled (with command-line flags to llvm-build) by the build system in order to be included in the auto-generated library dependency table used to build the llvm-config tool.

For end-users, this means the command "llvm-config -libs all" does not list libraries that are not built by the current configuration.

The proposed commit message is:

Make IntelJITEvents and OProfileJIT as optional libraries and add optional library support to the llvm-build tool:

-        Add new command line parameter to llvm-build: "--enable-optional-libraries"

-        Add handing of new llvm-build library type "OptionalLibrary"

-        Update Cmake and automake build systems to pass correct flags to llvm-build based on configuration

Thanks,
Dan

Daniel Malea daniel.malea at intel.com<mailto:daniel.malea at intel.com>
  Intel Embedded Computing, Debuggers & Libraries
  Intel Waterloo

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120416/f982f20c/attachment.html>