Skip to content

[TritonToLinalg](feat) Tag conv1d as mix mode - #1534

Closed
lowdy1 wants to merge 1 commit into
triton-lang:mainfrom
lowdy1:conv1d_mix
Closed

[TritonToLinalg](feat) Tag conv1d as mix mode#1534
lowdy1 wants to merge 1 commit into
triton-lang:mainfrom
lowdy1:conv1d_mix

Conversation

@lowdy1

@lowdy1 lowdy1 commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Since conv1d runs on the AI Cube unit, we tag it as existDot = true, so any kernel with al.conv1d is set as mix_mode instead of aiv.

New contributor declaration

  • I am not making a trivial change, such as fixing a typo in a comment.

  • I have written a PR description following these
    rules.

  • I have run pre-commit run --from-ref origin/main --to-ref HEAD.

  • Select one of the following.

    • I have added tests.
      • /test for lit tests
      • /unittest for C++ tests
      • /python/test for end-to-end tests
    • This PR does not need a test because FILL THIS IN.
  • Select one of the following.

    • I have not added any lit tests.
    • The lit tests I have added follow these best practices,
      including the "tests should be minimal" section. (Usually running Python code
      and using the instructions it generates is not minimal.)

@github-actions github-actions Bot added compiler Changes to C/C++ compiler backend (lib/, include/) python Changes to Python runtime or bindings ascend-backend Changes to the Ascend NPU backend labels Aug 13, 2026
@lowdy1 lowdy1 changed the title [feat] Tag conv1d as mix mode [TritonToLinalg](feat) Tag conv1d as mix mode Aug 13, 2026
@github-actions

github-actions Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

OpenCodeReview: Review complete: 0 finding(s) across 2 selected item(s).

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

Labels

ascend-backend Changes to the Ascend NPU backend compiler Changes to C/C++ compiler backend (lib/, include/) python Changes to Python runtime or bindings

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant