Ok… I wrote some CMake goop to enable sanitizers on a bootstrap build (I’ll 
clean up the patches and send those out separately). check-clang worked with 
asan and ubsan enabled.

Is that sufficient pre-commit testing?

I still need to rework the Makefile side of the patch to match Eric’s 
suggestion.

-Chris

> On Aug 14, 2015, at 11:14 AM, Justin Bogner <m...@justinbogner.com> wrote:
> 
> Chris Bieneman <be...@apple.com> writes:
>> beanz added a comment.
>> 
>> dblaikie,
>> 
>> I've run this through check-all with no issues.
> 
> Might want to bootstrap with sanitizers or something. Strict aliasing
> related bugs can be subtle.

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to