Skip to content

[AdvancedCompiler]Cat(cpp wrapper) - #733

Merged
StrongSpoon merged 5 commits into
flagos-ai:masterfrom
AdvancedCompiler:cat(cpp-wrapper)
Jul 9, 2025
Merged

[AdvancedCompiler]Cat(cpp wrapper)#733
StrongSpoon merged 5 commits into
flagos-ai:masterfrom
AdvancedCompiler:cat(cpp-wrapper)

Conversation

@AdvancedCompiler

Copy link
Copy Markdown
Collaborator

PR Category

Operator

Type of Change

New Feature

Description

CPP wrapper packaging for cat op.

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

@StrongSpoon StrongSpoon self-assigned this Jul 3, 2025

@StrongSpoon StrongSpoon left a comment

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.

Tests are very comprehensive and can be extended to python ut.

Comment thread include/flag_gems/operators.h Outdated
Comment thread lib/cat.cpp
Comment thread ctests/test_triton_cat.cpp Outdated
Comment thread CMakeLists.txt Outdated
FetchContent_Declare(TritonJIT
GIT_REPOSITORY https://github.com/iclementine/libtorch_example.git
GIT_REPOSITORY git@github.com:AdvancedCompiler/libtriton_jit.git
# GIT_REPOSITORY https://github.com/iclementine/libtorch_example.git

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.

I suggest using the original repository. If any code changes are made, please create a pull request to libtorch_example.

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.

Thanks for the review . Downloaded from advancedcompiler due to permission issues; no modifications were made to libtriton_jit. Fixed. : )

@StrongSpoon StrongSpoon left a comment

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.

lgtm

@StrongSpoon
StrongSpoon merged commit a0297cb into flagos-ai:master Jul 9, 2025
16 of 22 checks passed
nicelynice pushed a commit to nicelynice/FlagGems that referenced this pull request Feb 24, 2026
* updata cat cpp wrapper

* fix CMakelist.txt

* fix PR733 cat cpp wrapper review

* fix /CMakeLists.txt

---------

Co-authored-by: Ea760 <15236119052@163.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.

4 participants