Richard, <div><br></div><div>The sanitizer_common part of your patch looks good, thanks! </div><div><br></div><div>--kcc <br><br><div class="gmail_quote">On Wed, Oct 31, 2012 at 12:38 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">This LGTM for the short term, but I'd like someone with more<br>
familiarity with the sanitizer_common pieces to look over it too (see<br>
attached ubsan.patch).<br>
<br>
<br>
Slightly longer-term, we should move away from using fprintf and over<br>
to sanitizer_common's Printf implementation. There are four format<br>
specifiers which ubsan uses which Printf doesn't yet provide:<br>
<br>
  %08x<br>
  %lld<br>
  %llu<br>
  %Lg<br>
<br>
The attached patch implements the first three and ports ubsan over to<br>
using sanitizer_common's Printf. Two more things need doing before we<br>
can switch to this:<br>
<br>
 1) Implement Printf support for formatting of long double<br>
 2) Move the 'isatty' check into sanitizer_common, so it can do the<br>
right platform-specific magic and can respect calls to<br>
__sanitizer_set_report_path<br>
<br>
Once those are done, ubsan won't need to include any system headers<br>
any more. Seem reasonable?<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, Oct 30, 2012 at 10:22 AM, Jean-Daniel Dupas<br>
<<a href="mailto:devlists@shadowlab.org">devlists@shadowlab.org</a>> wrote:<br>
> Ping<br>
><br>
> Le 22 oct. 2012 à 12:51, Jean-Daniel Dupas <<a href="mailto:devlists@shadowlab.org">devlists@shadowlab.org</a>> a écrit :<br>
><br>
>> Thanks.<br>
>><br>
>> Look like you forget to "svn add" the new Makefile and unistd.h though.<br>
>><br>
>> Here is a rebased svn patch that include these files.<br>
>><br>
>> Le 22 oct. 2012 à 12:31, Jonathan Sauer <<a href="mailto:jonathan.sauer@gmx.de">jonathan.sauer@gmx.de</a>> a écrit :<br>
>><br>
>>> Hello,<br>
>>><br>
>>> I applied the patch to compiler-rt r166410 (darwin-ubsan.patch did not apply cleanly, most likely due<br>
>>> to changes made in the meantime), and compiler-rt compiles successfully as part of LLVM (i.e. as<br>
>>> described at <<a href="http://clang.llvm.org/get_started.html" target="_blank">http://clang.llvm.org/get_started.html</a>>) on OS X 10.6. I rebased your patches to r166410<br>
>>> and attached the new one (made from svn instead of git, though).<br>
>>><br>
>>><br>
>>> Jonathan<br>
>>><br>
>>> <ubsan.diff><br>
>>><br>
>>> _______________________________________________<br>
>>> cfe-commits mailing list<br>
>>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
>><br>
>> -- Jean-Daniel<br>
>><br>
>><br>
>><br>
>> <ubsan.diff>_______________________________________________<br>
>> cfe-commits mailing list<br>
>> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
>> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
> -- Jean-Daniel<br>
><br>
><br>
><br>
><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
><br>
</div></div></blockquote></div><br></div>