[PATCH 0/5] expand sin/cos and some cleanup

Jan Vesely jan.vesely at rutgers.edu
Sun Apr 20 13:57:50 PDT 2014


Hi,

only the first patch adds new functionality. The two followups just move
more expansion stuff to R600 file. I checked that all these ops have vector
instructions on SI.

I'm not sure what the policy is so I can send v2 with stuff moved back to
AMDGPUISelLowering.

The test are rather terse, but I'm not sure what influences instruction
ordering, and 4 SIN/COS instructions should be good enough sign that the 
expansion is happening.

regards,
Jan

Jan Vesely (5):
  R600: Expand vector sin and cos on r600.
  R600: Remove duplicate setting of SELECT expansion.
  R600: Move R600 scalar operations initialization to
    R600ISelLowering.cpp
  R600: Add vector COS test
  R600: Add vector SIN test

 lib/Target/R600/AMDGPUISelLowering.cpp | 21 ++-------------------
 lib/Target/R600/R600ISelLowering.cpp   | 23 +++++++++++++++++++++--
 test/CodeGen/R600/llvm.cos.ll          | 13 +++++++++++++
 test/CodeGen/R600/llvm.sin.ll          | 14 ++++++++++++++
 4 files changed, 50 insertions(+), 21 deletions(-)

-- 
1.9.0




More information about the llvm-commits mailing list