[Mlir-commits] [mlir] [mlir][mpi] Lowering Mpi To LLVM (PR #127053)

[Frank Schlimbach]

fschlimb wrote:

I thought a bit about the runtime option. I see only two ways of doing this in a reasonable way:

1. I found no way of adding  custom option to `--convert-to-llvm`. So if we want this to become an option to the pass, we need a separate pass (and not populate the generic  `--convert-to-llvm` pass). There seems to be some consensus that the current approach is preferred (and In #95524 @joker-eph explicitly asked for it). Any hint on how to add an option to `--convert-to-llvm` would be appreciated.
2. Use DLTI to annotate the IR. This could be something like the below. I prefer this solution, it is more flexible (e.g. one could write an extra pass wich inserts the DLTI according to an option to the pass) and keeps the integration into `--convert-to-llvm`.

```mlir
module attributes { mpi.dlti = #dlti.map<"MPI:Implemention" = "Intel"> } {
  %0 = mpi.....
}
```

Any thoughts on this?

https://github.com/llvm/llvm-project/pull/127053