On Mon, Jan 16, 2012 at 10:25 AM, Ruben Van Boxem
<[email protected]> wrote:
> 2012/1/16 Eli Friedman <[email protected]>
>>
>> On Mon, Jan 16, 2012 at 8:24 AM, Ruben Van Boxem
>> <[email protected]> wrote:
>> > Hi,
>> >
>> > Is there a specific reason why Clang does not have
>> > __builtin_labs
>> > __builtin_llabs
>>
>> No, probably just nobody got around to adding them.
>
>
> I had an attempt at adding the __builtin's mentioned above. I tested with
> current trunk attached patch. It just adds the two builtin names and puts it
> in the same codegen part as regular abs. I hope this is correct, I had a
> quick check of llabs with -2^61 and printf using "%lld".
>
> Thanks,
>
> Ruben
>
> PS: If patch is OK, please commit, I don't have commit rights.

r148340.

-Eli
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to