compiler-explorer / compiler-explorer/infra

mlir-opt vs mlir-translate ?

Open
#1,022 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
434
Forks
429
Avg merge
4d 21h
Merged PRs (30d)
38

Description

In the line referenced [1] , why do you use mlir-translate instead of mlir-opt ?
Background here (https://discourse.llvm.org/t/doc-mlir-translate-mlir-opt/60751)

I would try to make it mlir-opt in case it's a bug, as mlir-opt is similar to llvm-opt and is what used for testing the IR, which can be the main usage of CE.

[1] https://github.com/compiler-explorer/infra/blob/c5cc7e516b8c3017a026734f23b49753c96920dc/bin/yaml/cpp.yaml#LL756C7-L756C46

Contributor guide

Open the contributing guide

Research direction

Review bin/yaml/cpp.yaml at the referenced line and compare the roles of mlir-translate and mlir-opt using the linked Discourse discussion. Confirm which command Compiler Explorer should use for this MLIR case; done means the configuration uses the appropriate command or the issue has a documented rationale for keeping the current one.

Written by the indexing model from the issue text.

Assessment

Domain
compilers
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.