Skip to content

[mlir] [dxsa] Add dcl_tgsm_structured instruction#144

Merged
tagolog merged 1 commit into
access-softek:dxsa-mlirfrom
tagolog:dxsa-mlir-dcl_tgsm_structured
May 27, 2026
Merged

[mlir] [dxsa] Add dcl_tgsm_structured instruction#144
tagolog merged 1 commit into
access-softek:dxsa-mlirfrom
tagolog:dxsa-mlir-dcl_tgsm_structured

Conversation

@tagolog
Copy link
Copy Markdown

@tagolog tagolog commented May 26, 2026

Example:
dxsa.dcl_tgsm_structured <type = thread_group_shared_memory, components = 0, index = [0]>, 16, 64

@tagolog tagolog requested a review from asavonic May 26, 2026 00:04
Copy link
Copy Markdown
Contributor

@asavonic asavonic left a comment

Choose a reason for hiding this comment

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

LGTM.

let summary = "declares a reference to a region of shared memory space available to the shader's thread group";
let description = [{
The `dxsa.dcl_tgsm_structured` operation declares a reference
to a region of shared memory space available to the shader's thread group.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Same as in #138 - let's deabbreviate TGSM here.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed

@tagolog tagolog force-pushed the dxsa-mlir-dcl_tgsm_structured branch from b5513e1 to 9d89e38 Compare May 27, 2026 17:09
Example:
  dxsa.dcl_tgsm_structured <type = thread_group_shared_memory, components = 0, index = [0]>, 16, 64

Signed-off-by: Vladimir Shiryaev <tagolog@users.noreply.github.com>
@tagolog tagolog force-pushed the dxsa-mlir-dcl_tgsm_structured branch from 9d89e38 to dfda104 Compare May 27, 2026 17:36
@tagolog tagolog merged commit ebce8ee into access-softek:dxsa-mlir May 27, 2026
5 checks passed
@tagolog tagolog deleted the dxsa-mlir-dcl_tgsm_structured branch June 1, 2026 20:26
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