<br><br><div class="gmail_quote">On Tue, Jul 26, 2011 at 10:20 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@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 class="im"><br>
On Jun 21, 2011, at 8:05 AM, Kostya Serebryany wrote:<br>
<br>
> Hi,<br>
> What would be our next steps in getting ASan into the LLVM trunk?<br>
> I'd like to do it in two steps, first for the LLVM part with minimal tests and then for the run-time library and all tests.<br>
> The current ASan's source repository will probably stay the primary home for the run-time library and tests as we plan to use it in non-LLVM environments.<br>
><br>
<br>
</div>Hi Kostya,<br>
<br>
I haven't had a chance to look at your patch yet, I'm backed up on "big patches".  Did you see my review of the safecode patch here?<br>
<a href="http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20110718/124515.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20110718/124515.html</a><br>
<br>
I expect to have similar concerns and suggestions for your patch,<br></blockquote><div><br></div><div>Hi Chris, </div><div><br></div><div>Thanks for the reply. </div><div>Indeed, some of your comments to safecode patch apply here. </div>
<div><br></div><div>Codingstyle: I'll try to cleanup as much as I can today. </div><div>Do you have any lint-like tool that checks for llvm coding style (similar to <a href="http://google-styleguide.googlecode.com/svn/trunk/cpplint/">cpplint</a>)? </div>
<div><br></div><div>Documentation: everything is in the wiki. The main pages are:</div><div><a href="http://code.google.com/p/address-sanitizer/wiki/AddressSanitizer">http://code.google.com/p/address-sanitizer/wiki/AddressSanitizer</a></div>
<div><a href="http://code.google.com/p/address-sanitizer/wiki/AddressSanitizerAlgorithm">http://code.google.com/p/address-sanitizer/wiki/AddressSanitizerAlgorithm</a></div><div><br></div>"who is going to maintain": my team at Google (in particular, myself and Alexander, in CC) are highly motivated to keep this working. </div>
"Do you have particular clients": the Chromium project is a <a href="http://blog.chromium.org/2011/06/testing-chromium-addresssanitizer-fast.html">very active</a> client.<div><br></div>"The work can be decomposed into small and incremental pieces": <div>
   the <a href="http://code.google.com/p/address-sanitizer/source/browse/trunk/llvm/AddressSanitizer.cpp">LLVM part</a> is just 1000 LOC. If you still like it to be decomposed, we can do it in 3 parts: a) general instrumentation, b) redzones for stack, c) redzones for globals.<div>
<br></div><div><br></div><div>Thanks, </div><div><br></div><div>--kcc </div><div> </div><div><div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">

<font color="#888888"><br>
-Chris<br>
</font></blockquote></div><br></div></div></div>