efriedma added a comment.

> I was going to include the builtins too, but that exposes another bug (marked 
> here with FIXME) - the builtins are all defined 'const'.

Probably just need to change c->e in Builtins.def?

> This does make me curious about the use-case of libm-equivalent builtins. If 
> they are exactly identical to libm (including errno behavior), then why are 
> they needed in the first place?

It gets treated differently under -fno-builtin/-std=c89.


https://reviews.llvm.org/D39204



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to