[Mlir-commits] [mlir] [mlir][mesh] Mandatory Communicator (PR #133280)

[Frank Schlimbach]

Sergio =?utf-8?q?Sánchez_Ramírez?=,
Sergio =?utf-8?q?Sánchez_Ramírez?=,
Sergio =?utf-8?q?Sánchez_Ramírez?=,
Sergio =?utf-8?q?Sánchez_Ramírez?=,
Sergio =?utf-8?q?Sánchez_Ramírez?=,"Schlimbach, Frank"
 <frank.schlimbach at intel.com>,"Schlimbach, Frank" <frank.schlimbach at intel.com>
 =?utf-8?q?,?=Frank Schlimbach <frank.schlimbach at intel.com>,Frank
 Schlimbach <frank.schlimbach at intel.com>,"Schlimbach, Frank"
 <frank.schlimbach at intel.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/133280 at github.com>


fschlimb wrote:

> IMO and summarizing, it is save to lower a `!mpi.comm` to a `llvm.ptr`

I agree it is safe to lower as an intermediate type. The latest commit lets the type converter convert `!mpi.com` to `i64` and before calling MPI functions an appropriate cast/trunc is done.

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