[PATCH] D21431: [arm+x86] Make GNU variants behave like GNU w.r.t combining sin+cos into sincos.

[Daniel Sanders via llvm-commits]

dsanders added a comment.

In http://reviews.llvm.org/D21431#459909, @t.p.northover wrote:

> I think it's just an omission to. `sincos` is definitely in my local copy of libm.so. I say go for it! (And thanks for the work).
>
> Tim.


Thanks.

In http://reviews.llvm.org/D21431#459913, @t.p.northover wrote:

> Actually, it looks like it's in the androideabi too (since 2.3), so it might be a good idea to add that.
>
> Tim.


Is this combine beneficial for android at the moment? According to https://android.googlesource.com/platform/bionic/+/master/libm/sincos.c, android's sincos uses -O0 and just forwards the call to sin and cos. It looks like you'd be getting two calls for the price of three.


http://reviews.llvm.org/D21431