<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Thu, Mar 14, 2013 at 1:24 AM, Jordan Rose <span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><div class="im">
<div>On Mar 13, 2013, at 14:21 , Alexey Samsonov <<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>> wrote:</div><br></div><blockquote type="cite"><div style="letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
<div dir="ltr">Hi!<div class="gmail_extra"><br><div class="gmail_quote"><div class="im">On Wed, Mar 13, 2013 at 8:04 PM, Jordan Rose<span> </span><span dir="ltr"><<a href="mailto:jordan_rose@apple.com" target="_blank">jordan_rose@apple.com</a>></span><span> </span>wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">1) Does this need to be added to the Makefile build as well?<br>
</blockquote><div><br></div></div><div>Um, at first glance I can't see how to define that a test suite depends on a certain tool…</div><div></div></div></div></div></div></blockquote><div><br></div><div>Ah, I guess in Make we just depend on LLVM implicitly being built first.</div>
<div class="im"><br><blockquote type="cite"><div style="letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">2) Should this be conditional on having compiler-rt?<br></blockquote>
<div><br></div><div>No, llvm-symbolizer doesn't depend on compiler-rt. "ASan or MSan" mentioned in</div><div>the commit message are turned on in the compiler flags of host compiler.</div></div></div></div></div>
</blockquote><br></div></div><div>Oh, the host compiler, not the built compiler. My bad. (If this were more expensive, I'd say it should be conditional on building with asan/msan, but llvm-symbolizer is light enough that it's fine to have the rest of us building it too.)</div>
</div></blockquote><div><br></div><div style>I agree, first part of better support for running under ASan/MSan is being reviewed at <a href="http://llvm-reviews.chandlerc.com/D459">http://llvm-reviews.chandlerc.com/D459</a>. When we have a nice support in the build system, it would be easy to optionally set dependencies.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><span class=""><font color="#888888"><div>
<br></div><div>Jordan</div></font></span></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>