Skip to content

feat:add triton op and cpp wrapper for copy_ and to_copy - #1294

Merged
kiddyjinjin merged 1 commit into
flagos-ai:masterfrom
wlxjhyf:ops
Dec 30, 2025
Merged

feat:add triton op and cpp wrapper for copy_ and to_copy#1294
kiddyjinjin merged 1 commit into
flagos-ai:masterfrom
wlxjhyf:ops

Conversation

@wlxjhyf

@wlxjhyf wlxjhyf commented Dec 28, 2025

Copy link
Copy Markdown
Collaborator

PR Category

Op

Type of Change

New Feature

Description

add cpp wrapper for copy_ and to_copy

Issue

Progress

  • Change is properly reviewed (1 reviewer required, 2 recommended).
  • Change is responded to an issue.
  • Change is fully covered by a UT.

Performance

@CLAassistant

CLAassistant commented Dec 28, 2025

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@wlxjhyf
wlxjhyf force-pushed the ops branch 2 times, most recently from e6fefef to 4a41f0a Compare December 28, 2025 14:58
@tengqm

tengqm commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

@wlxjhyf Please check your CLA status. Signing the CLA is required before your PR is considered as a merge candidate.

Comment thread lib/copy.cpp Outdated
return out;
}

void copy_(const at::Tensor& dst, const at::Tensor& src) {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

the schema is Tensor& copy_ (Tensor & self, const Tensor& src, bool non_blocking)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

copy_ and to_copy are implemented to match PyTorch’s operator schemas.

@wlxjhyf
wlxjhyf force-pushed the ops branch 6 times, most recently from 60ed2ff to 792d56b Compare December 30, 2025 05:33
@kiddyjinjin
kiddyjinjin merged commit d469d3b into flagos-ai:master Dec 30, 2025
11 of 13 checks passed
nicelynice pushed a commit to nicelynice/FlagGems that referenced this pull request Feb 24, 2026
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.

4 participants