On Wed, Mar 18, 2009 at 10:34 AM, Mike Stump <span dir="ltr"><<a href="mailto:mrs@apple.com">mrs@apple.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Mar 18, 2009, at 10:06 AM, Daniel Dunbar wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
This isn't right; the Driver library shouldn't have these things embedded in it. The reason I left it blank was because I was going to do try and provide a proper version number derived from SVN (at some point).<br>

</blockquote>
<br></div>
Derived how?  During the build, there isn't necessarily any svn there, so one cannot use svn in any way to derive it.  One can use the various LLVM #defines, but these don't include the tag information when someone builds a non-standard tag and I would like to have that tag information.  Anyway, I didn't see any constructive criticism about what you saw and what you would rather see instead.  Currently it prints something like:<br>

<br>
ccc version 1.0 (<a href="http://llvm.org/svn/llvm-project/cfe/trunk" target="_blank">http://llvm.org/svn/llvm-project/cfe/trunk</a>)<br>
<br>
which I think is fine.  If someone tags clang, say for an llvm release, it will have the release tag.  If we want to add the svn revision information, a la gcc, we could do that, but it is a harder to get at and isn't always available (svn export say).<br>
<div class="im"></div></blockquote><div><br>Yes, this is what I meant.<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div class="im">

<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Also, the use of TARGET_TRIPLE isn't necessary; the Driver already knows what host it is on (DefaultHostTriple),<br>
</blockquote>
<br></div>
Yeah, the only problem with that is I don't actually want to print the host information.  Now, if you had DefaultTargetTriple, that would be closer to what I'd want.<br>
</blockquote></div><br>The default tool chain has this information (in pieces currently, I will probably introduce an explicit Triple object at some point); it can be accessed through the Compilation structure.<br><br> - Daniel<br>
<br>