Skip to content

Conversation

@areusch
Copy link
Contributor

@areusch areusch commented Mar 22, 2023

Fixes a bug where this stacktrace is generated:

  File "/opt/scorecard/relax/python/tvm/contrib/cutlass/gen_tensor_op.py", line 189, in generate_sm75_tensor_op_1688
    return generate_sm50_simt(
  File "/opt/scorecard/relax/python/tvm/contrib/cutlass/gen_tensor_op.py", line 119, in generate_sm50_simt
    return generate_tensor_op_common(
  File "/opt/scorecard/relax/python/tvm/contrib/cutlass/gen_tensor_op.py", line 71, in generate_tensor_op_common
    out = op_creator(tile_descriptions, data_type, alignment_constraints)
  File "/opt/scorecard/relax/python/tvm/contrib/cutlass/gen_gemm.py", line 151, in enumerate_gemm_operators
    op.procedural_name(),
  File "/opt/scorecard/relax/python/tvm/contrib/cutlass/gemm_operation.py", line 110, in procedural_name
    opcode_class_name = OpcodeClassNames[self.tile_description.math_instruction.opcode_class]
KeyError: <MathOperation.multiply_add: 1>

@masahi @jwfromm

@tvm-bot
Copy link
Collaborator

tvm-bot commented Mar 22, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@tqchen tqchen changed the title Also include output dtype in simt MathInstruction [Unity] Also include output dtype in simt MathInstruction Mar 22, 2023
@tqchen tqchen merged commit 4e46ad4 into apache:unity Mar 22, 2023
tqchen pushed a commit that referenced this pull request Apr 1, 2023
Also include output dtype in simt MathInstruction.
tqchen pushed a commit that referenced this pull request Apr 1, 2023
Also include output dtype in simt MathInstruction.
tqchen pushed a commit that referenced this pull request Apr 1, 2023
Also include output dtype in simt MathInstruction.
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