[PATCH] Add FreeBSD support to Ubsan
Ivan A. Kosarev
ivan at ivan-labs.com
Mon Feb 24 07:21:55 PST 2014
Fixed. Thanks.
--
On 02/24/2014 05:23 PM, Alp Toker wrote:
>
> 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
>
More information about the llvm-commits
mailing list