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

#8815: Updating bcast to no longer use run with auto format #9360

Closed
wants to merge 1 commit into from

Conversation

eyonland
Copy link
Contributor

Initially just trying to get rid of the run with auto format on bcast.

Tensor zero_like = mk_zero_tensor_like(reference_tensor, output_mem_config);
Tensor result = bcast(zero_like, k, BcastOpMath::ADD, BcastOpDim::HW, output_mem_config);
k.deallocate();
Copy link
Member

Choose a reason for hiding this comment

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

when will deallocation happen if not called manually?

@@ -222,6 +222,18 @@ Tensor triu(
return tt::tt_metal::triu(input_tensor, diag, memory_config.value_or(input_tensor.memory_config()));
}

// TODO
Copy link
Member

Choose a reason for hiding this comment

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

Can we remove this?

@eyonland eyonland force-pushed the ttnn-9302-more-unary-composite-ops branch from 6012f8a to 19833a0 Compare June 12, 2024 21:16
@eyonland eyonland force-pushed the ttnn-9302-more-unary-composite-ops branch from 19833a0 to 80800fc Compare June 12, 2024 21:29
@github-actions github-actions bot closed this Dec 18, 2024
@github-actions github-actions bot deleted the ttnn-9302-more-unary-composite-ops branch December 18, 2024 00:28
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