[Mlir-commits] [mlir] [mlir][LLVM] Add support for `ptrtoaddr` (PR #185104)

[Tobias Gysi]

Markus =?utf-8?q?Böck?= <markus.boeck02 at gmail.com>,
Markus =?utf-8?q?Böck?= <markus.boeck02 at gmail.com>,
Markus =?utf-8?q?Böck?= <markus.boeck02 at gmail.com>,
Markus =?utf-8?q?Böck?= <markus.boeck02 at gmail.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/185104 at github.com>


================
@@ -1283,6 +1283,24 @@ llvm.func @intpointerconversion(%arg0 : i32) -> i32 {
   llvm.return %2 : i32
 }
 
+// CHECK-LABEL: @addrpointerconversion_scalar
+// CHECK-SAME: %[[ARG0:[[:alnum:]]+]]
+llvm.func @addrpointerconversion_scalar(%arg0 : !llvm.ptr) -> i64 {
+  // CHECK: %[[PTR:.*]] = ptrtoaddr ptr %[[ARG0]] to i64
+  // CHECK-NEXT: ret i64 %[[PTR]]
----------------
gysit wrote:

```suggestion
// CHECK:      %[[PTR:.*]] = ptrtoaddr ptr %[[ARG0]] to i64
// CHECK-NEXT: ret i64 %[[PTR]]
```
ultra nit: I would probably mimic the style in the rest of the file (avoiding hardcoding the ssa values of course)

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