[Mlir-commits] [mlir] [mlir][DataLayout] Add a default memory space entry to the data layout. (PR #127416)

[Christian Ulmann]

================
@@ -318,7 +318,8 @@ combineOneSpec(DataLayoutSpecInterface spec,
            "unexpected data layout entry for built-in type");
 
     auto interface = cast<DataLayoutTypeInterface>(typeSample);
-    if (!interface.areCompatible(entriesForType.lookup(kvp.first), kvp.second))
+    if (!interface.areCompatible(entriesForType.lookup(kvp.first), kvp.second,
----------------
Dinistro wrote:

Sorry for the late reply.
I guess the pointer type is somewhat special in that regard as this information is only accessible when going to the spec. Considering this, and that there might be other obscure use cases for this, I think this is probably acceptable.

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