[PATCH] Add FreeBSD support to Ubsan

Alp Toker alp at nuanti.com
Mon Feb 24 05:23:47 PST 2014


On 24/02/2014 13:06, Kostya Serebryany wrote:
>    LGTM
>    Nick?
>   
>   // For now, only support linux and darwin. Other platforms should be easy to
>   // add, and probably work as-is.
> -#if !defined(__linux__) && !defined(__APPLE__)
> +#if !defined(__linux__) && !defined(__FreeBSD__) && !defined(__APPLE__)
>   #error "UBSan not supported for this platform!"
>   #endif

Probably want to change the comment too, or remove the "only support" 
part so it doesn't have to be kept in sync?

Alp.


>
> http://llvm-reviews.chandlerc.com/D2867
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-- 
http://www.nuanti.com
the browser experts




More information about the llvm-commits mailing list