address reviewer comments

Author: fabianmcg

mlir/include/mlir/Interfaces/DataLayoutInterfaces.td

@@ -657,7 +657,8 @@ def DataLayoutTypeInterface : TypeInterface<"DataLayoutTypeInterface"> {
                "is, that `newLayout` spec entries can be nested in an op with "
                "`oldLayout` spec entries. `newSpec` and `identified` are"
                "provided to further query data from the combined spec, e.g.,"
-               "the default address space.",
+               "the default address space. TODO: Revisit this method once"
+               "https://github.com/llvm/llvm-project/issues/130321 gets solved",
       /*retTy=*/"bool",
       /*methodName=*/"areCompatible",
       /*args=*/(ins "::mlir::DataLayoutEntryListRef":$oldLayout,

mlir/lib/Dialect/DLTI/DLTI.cpp

@@ -340,7 +340,8 @@ combineOneSpec(DataLayoutSpecInterface spec,
            "unexpected data layout entry for built-in type");
 
     auto interface = cast<DataLayoutTypeInterface>(typeSample);
-    // TODO: Revisit this method and call once issue #130321 gets resolved.
+    // TODO: Revisit this method and call once
+    // https://github.com/llvm/llvm-project/issues/130321 gets resolved.
     if (!interface.areCompatible(entriesForType.lookup(kvp.first), kvp.second,
                                  spec, entriesForID))
       return failure();