search

accel-sim / gpu-app-collection

A repository where GPU applications are aggregated using a common build flow that supports multiple CUDA versions.
44 stars 32 forks source link

update cutlass to build with name used in accelwattch, delete unused … #22

Closed barnes88 closed 2 months ago

barnes88 commented 2 months ago

The cutlass makefile was failing because it was trying to copy files that didn't exist. This fixes that issue, I also removed the cutlass-bench-test folder because none of the files were being used and it seems like it was committed by mistake. I traced, tested, and verified the first 50 kernels runs using the QV100-SASS config run in simulation without issue.

JRPan commented 2 months ago

cutlass-bench-test was a submodule, I think, Shreya added that. So, if I'm correct, you basically reversed her PR. haha

barnes88 commented 2 months ago

cutlass-bench-test was a submodule, I think, Shreya added that. So, if I'm correct, you basically reversed her PR. haha

No this is the commit that added it as a submodule: https://github.com/accel-sim/gpu-app-collection/commit/0c9dc3a03d54fa3b0de14358ac23575a9686c698 which is unchanged. The whole point of submodules is so you don't have to add all the files individually to the repo, so these files were redundant