Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the sgemm_nt_1_sycl PoC #15

Merged

Conversation

aacostadiaz
Copy link
Collaborator

This PR removes the sgemm_nt_1_sycl PoC implemented before the new examples for Cute were added in Cutlass 3.5.

It also fixes some build issues introduced in PR#13 and addresses some issues with code style.

Copy link
Collaborator

@mehdi-goli mehdi-goli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I figured out we can remove all the no needed flag

include/cute/config.hpp Outdated Show resolved Hide resolved
include/cute/config.hpp Outdated Show resolved Hide resolved
include/cute/config.hpp Outdated Show resolved Hide resolved
include/cute/config.hpp Outdated Show resolved Hide resolved
include/cute/config.hpp Outdated Show resolved Hide resolved
include/cute/arch/mma_sm80.hpp Outdated Show resolved Hide resolved
include/cute/arch/mma_sm80.hpp Outdated Show resolved Hide resolved
include/cute/arch/mma_sm80.hpp Outdated Show resolved Hide resolved
include/cute/arch/mma_sm80.hpp Outdated Show resolved Hide resolved
include/cute/arch/mma_sm80.hpp Outdated Show resolved Hide resolved
@aacostadiaz aacostadiaz merged commit a00e437 into codeplaysoftware:sycl-develop Apr 4, 2024
@aacostadiaz aacostadiaz deleted the aacosta/remove_poc branch April 4, 2024 16:20
mehdi-goli added a commit that referenced this pull request Apr 4, 2024
* Remove sgemm_nt_1 PoC

* Fix build issues

* Fix code style format

* Remove ENABLE_NVPTX flag

* Update include/cute/util/debug.hpp

Co-authored-by: Mehdi Goli <mehdi.goli@codeplay.com>

* Cosmetic

---------

Co-authored-by: Mehdi Goli <mehdi.goli@codeplay.com>
mehdi-goli added a commit that referenced this pull request Apr 4, 2024
* Updating README-sycl.md to capture the 3.5 modifications

* Update README-sycl.md

Co-authored-by: aacostadiaz <alejandro.acosta@codeplay.com>

* Remove the sgemm_nt_1_sycl PoC (#15)

* Remove sgemm_nt_1 PoC

* Fix build issues

* Fix code style format

* Remove ENABLE_NVPTX flag

* Update include/cute/util/debug.hpp

Co-authored-by: Mehdi Goli <mehdi.goli@codeplay.com>

* Cosmetic

---------

Co-authored-by: Mehdi Goli <mehdi.goli@codeplay.com>

* Applying the comments

---------

Co-authored-by: aacostadiaz <alejandro.acosta@codeplay.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants