[Mlir-commits] [mlir] [MLIR][GPU] Update serializeToObject to use SerializedObject wrapper (PR #176697)

[Zichen Lu]

================
@@ -747,6 +771,11 @@ NVVMTargetAttrImpl::serializeToObject(Attribute attribute, Operation *module,
   if (isaToBinaryTimeInMs.has_value())
     module->setAttr("ISAToBinaryTimeInMs",
                     builder.getI64IntegerAttr(*isaToBinaryTimeInMs));
+  StringRef isaCompilerLog = serializer.getISACompilerLog();
+  if (!isaCompilerLog.empty())
+    module->setDiscardableAttr("ISACompilerLog",
+                               builder.getStringAttr(isaCompilerLog));
----------------
MikaOvO wrote:

@joker-eph Thanks for the commit! I've adopted your approach. Could you please review when you have time?

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