[ blas/OpenCL ] SGEMM OpenCL kernels added #2648
Conversation
s-debadri
requested review from
myungjoo,
jijoongmoon,
again4you,
jaeyun-jung,
leemgs,
wooksong,
helloahn,
kparichay,
gichan-jang,
anyj0527,
zhoonit,
lhs8928,
songgot,
jihochu,
DonghakPark,
SeoHyungjun,
baek2sm,
skykongkong8,
djeong20,
EunjuYang and
a team
as code owners
|
📝 TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2648. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/. |
There was a problem hiding this comment.
@s-debadri, 💯 All CI checkers are successfully verified. Thanks.
| GEN_TEST_INPUT(A, ((i * (batch * height * channel) + j * (batch * height) + | ||
| k * (width) + l + 1) % | ||
| MOD) * | ||
| alpha); | ||
| GEN_TEST_INPUT_B(B, ((i * (batch * height_b * channel) + | ||
| j * (batch * height_b) + k * (width_b) + l + 1) % | ||
| MOD) * | ||
| alpha); |
There was a problem hiding this comment.
Was it intended to skip testing on fp16?
There was a problem hiding this comment.
Running tests on fp32 only as of now
s-debadri
force-pushed
the
gpu_sgemm
branch
2 times, most recently
from
97cea3d to
d156796
Compare
There was a problem hiding this comment.
@s-debadri, 💯 All CI checkers are successfully verified. Thanks.
s-debadri
changed the title
| @@ -44,7 +44,7 @@ TEST(blas_kernels, dotCL_sgemv) { | |||
| int width = 768; | |||
|
|
|||
| int height_b = 768; | |||
| int width_b = 96000; | |||
| int width_b = 2048; | |||
There was a problem hiding this comment.
is there any particular reason to change this?
There was a problem hiding this comment.
No reason as such. Was testing with different values. Might have changed it to run this test faster to check other test results quickly.
Added all possible OpenCL kernels for SGEMM Added unit tests Signed-off-by: Debadri Samaddar <[email protected]>
There was a problem hiding this comment.
@s-debadri, 💯 All CI checkers are successfully verified. Thanks.
Added SGEMM OpenCL kernels for the following:
noTranstransAtransBtransABSelf evaluation:
Signed-off-by: Debadri Samaddar [email protected]