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

fix Copy_Traits<SM80_CP_ASYNC_*_ZFILL> without definition of with(pred) function #1887

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

CalebDu
Copy link
Contributor

@CalebDu CalebDu commented Oct 20, 2024

Definition with(pred) func of Copy_Traits<SM80_CP_ASYNC_*_ZFILL> is placed in wrong class, it causes pred passed to SM80_CP_ASYNC_*_ZFILL always is false and cp_async always fill all zeros.

…_Traits<SM80_CP_ASYNC_CACHEALWAYS_ZFILL<S,D>> without definition of with(pred) function ,

It causes  pred in CP_ASYNC_ZFILL always be false and fill zeros.
@CalebDu
Copy link
Contributor Author

CalebDu commented Nov 3, 2024

@ccecka

@ccecka
Copy link

ccecka commented Nov 4, 2024

I have my own MR of fixes for copy and this is included in that, including tracking down all of the places where SM80_CP_ASYNC_* is chosen instead of the intended ZFILL variant.

That will be getting merged soon and pushed out publicly, I hope.

@CalebDu
Copy link
Contributor Author

CalebDu commented Nov 5, 2024

I have my own MR of fixes for copy and this is included in that, including tracking down all of the places where SM80_CP_ASYNC_* is chosen instead of the intended ZFILL variant.

That will be getting merged soon and pushed out publicly, I hope.

I get it. looking forward your fix releasing soon

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.

2 participants