This is an archive of the discontinued LLVM Phabricator instance.

[mlir][Transform] Add an example of a preloaded reference lowering to LLVM
ClosedPublic

Authored by nicolasvasilache on Sep 1 2023, 8:51 AM.

Diff Detail

Event Timeline

Herald added a reviewer: dcaballe. · View Herald Transcript
Herald added a project: Restricted Project. · View Herald Transcript
nicolasvasilache requested review of this revision.Sep 1 2023, 8:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 1 2023, 8:51 AM
mlir/test/Dialect/LLVM/lower-to-llvm-e2e.mlir
57

@ftynse I think we could simplify some genuflexions here around wrapping payload in a transform.target_tag="payload" if we had a transform.module op.
Thoughts ?

ftynse accepted this revision.Sep 4 2023, 4:55 AM

@ftynse I think we could simplify some genuflexions here around wrapping payload in a transform.target_tag="payload" if we had a transform.module op.

Thoughts ?

It would make it impossible to target a single op, rather than a region attached to the module op. It could also interfere with verification rules in some cases (e.g. when we cannot just inject random module around an op, such as GPU module/kernel verification). I really don't see a problem with attaching an attribute for debugging purposes.

mlir/test/Dialect/LLVM/lower-to-llvm-e2e.mlir
57

@ftynse I think we could simplify some genuflexions here around wrapping payload in a transform.target_tag="payload" if we had a transform.module op.
Thoughts ?

This revision is now accepted and ready to land.Sep 4 2023, 4:55 AM