[PATCH] D41659: Implementing missing trigonometric optimizations

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 03:21:19 PST 2018


davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.

Meta point: It's unclear to me whether this is something profitable to implement.
Sure, you can probably take a textbook and implement all the possible identities written there, but, what's the point?

I looked and it seems other compilers (most notably, GCC) don't implement at least the first transformation you implemented (I didn't bother to check the others).
https://godbolt.org/g/kKP5PW

tl;dr: what's your motivation?


Repository:
  rL LLVM

https://reviews.llvm.org/D41659





More information about the llvm-commits mailing list