[PATCH] D12036: We shouldn't need to pass -fno-strict-aliasing when building clang with clang.

Chris Bieneman via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 14 14:52:04 PDT 2015


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 <mail at justinbogner.com> wrote:
> 
> Chris Bieneman <beanz at 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.



More information about the cfe-commits mailing list