[PATCH] D41659: Implementing missing trigonometric optimizations

[ADIL ARUN DANGUI via Phabricator via llvm-commits]

cs15btech11041 added a comment.

In https://reviews.llvm.org/D41659#965694, @fhahn wrote:

> Thanks for your patch!
>
> Usually it is easier to review smaller patches. Splitting it up into 4 patches for the 4 different cases you implemented would make it slightly easier to reason about the 4 unrelated transformation in isolation.


Yes. Thank you for the suggestions. I have updated the diff to incorporate the changes you suggested. And submitted different test case for each optimization in isolation. Hope it has made it easier to review the patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D41659