Hi Daniel, <div><br></div><div>Chris suggested to talk to you about committing the AddressSanitizer (asan) run-time into the llvm tree (<span style="background-color: rgb(255, 255, 255); color: rgb(34, 34, 34); font-family: 'Lucida Grande', 'Lucida Sans Unicode', Arial, Verdana, Helvetica, sans-serif; line-height: 19px; ">llvm-project/compiler-rt</span>). </div>
<div><br></div><div>Questions: </div><div>- What is the preferred name for the directory? (asan? libasan? address_sanitizer? AdressSanitizer?)</div><div>- Should the asan run-time use cmake, or just make, or what? The build is a bit tricky, especially for tests. We currently use make.</div>
<div>- How would you suggest to do the code review? </div><div>   The code of the run-time is ~5 KLOC. <a href="http://code.google.com/p/address-sanitizer/source/browse/#svn%2Ftrunk%2Fasan">http://code.google.com/p/address-sanitizer/source/browse/#svn%2Ftrunk%2Fasan</a></div>
<div>   The Apple-specific part may make some Apple experts cry (or maybe not). </div><div>   The code uses google's coding style, which is similar, but not equivalent to the LLVM's one. We check it using cpplint before commits. </div>
<div>   LLVM license is used. </div><div>   The tests are ~2.5 KLOC; most of the tests require the clang compiler with -faddress-sanitizer switch. </div><div>   If possible, I'd prefer not to review the patch in a usual way (too big), but instead have the comments to the files in <a href="http://code.google.com/p/address-sanitizer/source/browse/#svn%2Ftrunk%2Fasan">http://code.google.com/p/address-sanitizer/source/browse/#svn%2Ftrunk%2Fasan</a></div>
<div>   Or, alternatively, commit everything as is and then iterate over individual files. </div><div>- The run-time library uses a couple of third_party files (BSD license) to parse process mapping.</div><div>  W/o these files asan will work, but will not produce symbolizable traces. </div>
<div>  Can we include these files as as in a separate directory? (They may be useful for other tools as well)</div><div>  <a href="http://code.google.com/p/address-sanitizer/source/browse/trunk/asan/sysinfo.cc">http://code.google.com/p/address-sanitizer/source/browse/trunk/asan/sysinfo.cc</a></div>
<div>- Yet another piece of third-party code (mit license) is used for Apple-specific work (function overriding). Same question as above apply. </div><div>   <a href="http://code.google.com/p/address-sanitizer/source/browse/trunk/asan/mach_override.c">http://code.google.com/p/address-sanitizer/source/browse/trunk/asan/mach_override.c</a></div>
<div>- test-specific: can I rely on gtest being installed? (fresh version is required).</div><div><br></div><div>Thanks, </div><div><br></div><div>--kcc </div><div><br></div><div><br></div>