Skip to content

Conversation

@dajunHuang
Copy link

The udpate function of device GemmUniversal's column-major specialization has a workspace parameter, and it's passed to underlying_operator_:

Status update(Arguments const &args, void *workspace = nullptr) {
return underlying_operator_.update(to_underlying_arguments(args), workspace);
}

But the update function of the underlying_operator_, that is, GemmUniversalBase, don't have this paramerter:
Status update(Arguments const &args)
{
CUTLASS_TRACE_HOST("GemmUniversalBase()::update()");
params_.update(args);
return Status::kSuccess;
}

This leads to compile error when using the column-major specialization, so I removed it.

@github-actions
Copy link

This PR has been labeled inactive-30d due to no recent activity in the past 30 days. Please close this PR if it is no longer required. Otherwise, please respond with a comment indicating any updates. This PR will be labeled inactive-90d if there is no activity in the next 60 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant