[Mlir-commits] [mlir] [MLIR][NVVM] Return ISA compiler log via ObjectAttr properties (PR #176697)

[Mehdi Amini]

================
@@ -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));
----------------
joker-eph wrote:

I don't quite get the complication with updating the API actually?

See here: https://github.com/llvm/llvm-project/commit/2ddea83d75345dd1986454ac28ef69363f3e83b9

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