<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 15, 2014 at 9:26 PM, Nick Kledzik <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div class="im"><div>On Jan 15, 2014, at 2:48 PM, Sean Silva wrote:</div></div>
<blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="im">On Wed, Jan 15, 2014 at 4:12 PM, <a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a> <span dir="ltr"><<a href="mailto:kledzik@apple.com" target="_blank">kledzik@apple.com</a>></span> wrote:<br>

</div><blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">
<br>
<br>
================<br>
Comment at: include/llvm-c/lto.h:201<br>
@@ -201,2 +201,2 @@<br>
<br>
<br>
----------------<br></div>
When you add new APIs to <llvm-c/lto.h> be sure to bump up LTO_API_VERSION, so that clients to do compile time configuration checks if the API exists.<br></blockquote><div class="im"><div><br></div><div>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?).</div>
</div></div></div></div></blockquote><br></div><div>Yes.  ld64 has lots of:</div><div>  #If LTO_API_VERSION > 5</div><div>    // use new lot api</div><div>  #endif</div><div>in its source code to enable it to build with lots of different versions of libLTO.dylib</div>
</div></blockquote><div><br></div><div>Ok, I just am not seeing the usual "Since version ...." mentions in the documentation.</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><span class="HOEnZb"><font color="#888888"><div><br></div><div>-Nick</div><div><br></div></font></span></div>
</blockquote></div><br></div></div>