[PATCH] Add an LTO interface to parse metadata and extract linker options

Sean Silva silvas at purdue.edu
Wed Jan 15 14:48:27 PST 2014


On Wed, Jan 15, 2014 at 4:12 PM, kledzik at apple.com <kledzik at apple.com>wrote:

>
>
> ================
> Comment at: include/llvm-c/lto.h:201
> @@ -201,2 +201,2 @@
>
>
> ----------------
> When you add new APIs to <llvm-c/lto.h> be sure to bump up
> LTO_API_VERSION, so that clients to do compiler configuration checks if the
> API exists.
>

We don't seem to document when features were introduced, so I don't see the
point in this (do we expect clients to go searching through our version
control history?). In practice, ./configure scripts and CMake use "does
this function exist" checks to directly test the existence of the function
(maybe you were talking about doing this purely inside an #ifdef in the
source code?).

-- Sean Silva


>
>
> http://llvm-reviews.chandlerc.com/D2343
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140115/057e1065/attachment.html>


More information about the llvm-commits mailing list