This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [3.4-BIB] Add trunc/round/floor/ceil/rint builtins


> Jan Hubicka <jh@suse.cz> writes:
> 
> [...]
> 
> | > | I see. Man page claims:
> | > | CONFORMING TO
> | > |        The floor() function conforms to SVID 3, POSIX, BSD 4.3, ISO
> | > |        9899.  The other functions are from C99.
> | > | 
> | > | So I will disable the builtins for non c++ except for floor, right?
> | > 
> | > If they are seen as __builtin_xxx() in source code then I don't think
> | > there is any need to disable them for C++.
> | I meant c99.
> 
> OK.  But then, I don't understand why you would want to disable it for
> C89 (the same reason(s) should apply to C++ equally).
In case rint is defined only by C99 and C89 don't even reserve it, I
have to.  In case it is reserved, my previous patch was correct modulo
the documentation bit.

I don't have standard here, can you check?

Honza
> 
> | I believe the behaviour is that they will be __bulitin_xxx but not xxx.
> 
> then, I have no problem with it.
> 
> -- Gaby


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]