Skip to content

[mlir][dxsa] Add dcl_input and dcl_output instructions#132

Open
tagolog wants to merge 2 commits into
access-softek:dxsa-mlirfrom
tagolog:dxsa-mlir-dcl_input-and-dcl_output
Open

[mlir][dxsa] Add dcl_input and dcl_output instructions#132
tagolog wants to merge 2 commits into
access-softek:dxsa-mlirfrom
tagolog:dxsa-mlir-dcl_input-and-dcl_output

Conversation

@tagolog
Copy link
Copy Markdown

@tagolog tagolog commented May 13, 2026

Example:
dxsa.dcl_input %v0
dxsa.dcl_output %o0

Example:
  dxsa.dcl_input %v0
  dxsa.dcl_output %o0

Signed-off-by: Vladimir Shiryaev <tagolog@users.noreply.github.com>
@tagolog tagolog requested review from asavonic and asl May 13, 2026 21:23
Encode the operand as an inline attribute instead of separate SSA values.

Example:
  dxsa.dcl_input <type = input, components = 4, mask = <x>, index = [0]>
  dxsa.dcl_output <type = output, components = 4, mask = <x, y, z, w>, index = [0]>

Signed-off-by: Vladimir Shiryaev <tagolog@users.noreply.github.com>
let assemblyFormat = "$value";
}

def DXSA_InlineOperandType_Temp : I32EnumAttrCase<"temp", 0>;
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.

I think it would be great if we can use these enums for regular operands as well.

auto loc = getLocation();

if (isImmOperand(*token))
return emitError(loc, "non-immediate operand is not supported yet");
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.

Non-immediate or immediate? Now it is inconsistent with isImmOperand(*token) condition.

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