<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">On Thu, Aug 8, 2013 at 3:48 AM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Perhaps we should lazily construct and store a SanitizerArgs instance on the Driver, and share it between the various places that want to inspect the sanitizer arguments?<br></blockquote><div><br></div><div>That's what first comes to mind. If it fits in the driver architecture and doesn't add unwanted global state, I can implement this.</div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  I don't think there's any good reason that the SanitizerArgs implementation mostly lives in the header. Since it's only included into two .cpp files, I don't think it matters much. Fix if you like :-)<br>
</blockquote><div><br></div><div>Okay, will do.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://llvm-reviews.chandlerc.com/D1300" target="_blank">http://llvm-reviews.chandlerc.com/D1300</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div>
</div></div>