[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