[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