Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Codegen] Update translation_info attribute assembly format. NFC. #19107

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

kuhar
Copy link
Member

@kuhar kuhar commented Nov 12, 2024

Drop the custom assembly format from the
DispatchLoweringPassPipelineAttr so that it can be exposed via python bindings.

The new format is:
#iree_codegen.translation_info<pipeline = X ...>

I updated the project files with the following commands:

sd -F 'iree_codegen.translation_info<' 'iree_codegen.translation_info<pipeline = ' ./**/*.mlir
sd -F 'translation_info = <' 'translation_info = #iree_codegen.translation_info<pipeline = ' ./**/*.mlir

Drop the custom assembly format from the
`DispatchLoweringPassPipelineAttr` so that it can be exposed via python
bindings.

The new format is:
`#iree_codegen.translation_info<pipeline = X ...>`

I updated the project files with the following commands:
```
sd -F 'iree_codegen.translation_info<' 'iree_codegen.translation_info<pipeline = ' ./**/*.mlir
sd -F 'translation_info = <' 'translation_info = #iree_codegen.translation_info<pipeline = ' ./**/*.mlir
```

Signed-off-by: Jakub Kuderski <[email protected]>
Copy link
Contributor

@MaheshRavishankar MaheshRavishankar left a comment

Choose a reason for hiding this comment

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

Thanks Jakub. This was annoying for sure

@kuhar kuhar merged commit d1a991c into iree-org:main Nov 12, 2024
34 of 36 checks passed
Groverkss pushed a commit to Groverkss/iree that referenced this pull request Dec 1, 2024
…ee-org#19107)

Drop the custom assembly format from the
`DispatchLoweringPassPipelineAttr` so that it can be exposed via python
bindings.

The new format is:
`#iree_codegen.translation_info<pipeline = X ...>`

I updated the project files with the following commands:
```
sd -F 'iree_codegen.translation_info<' 'iree_codegen.translation_info<pipeline = ' ./**/*.mlir
sd -F 'translation_info = <' 'translation_info = #iree_codegen.translation_info<pipeline = ' ./**/*.mlir
```
giacs-epic pushed a commit to giacs-epic/iree that referenced this pull request Dec 4, 2024
…ee-org#19107)

Drop the custom assembly format from the
`DispatchLoweringPassPipelineAttr` so that it can be exposed via python
bindings.

The new format is:
`#iree_codegen.translation_info<pipeline = X ...>`

I updated the project files with the following commands:
```
sd -F 'iree_codegen.translation_info<' 'iree_codegen.translation_info<pipeline = ' ./**/*.mlir
sd -F 'translation_info = <' 'translation_info = #iree_codegen.translation_info<pipeline = ' ./**/*.mlir
```

Signed-off-by: Giacomo Serafini <[email protected]>
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