[Mlir-commits] [mlir] [mlir][target][nvvm] Perf by stage and store into properties (PR #126178)

[Mehdi Amini]

================
@@ -220,6 +221,16 @@ class NVPTXSerializer : public SerializeGPUModuleBase {
   std::optional<SmallVector<char, 0>>
   moduleToObject(llvm::Module &llvmModule) override;
 
+  /// Get LLVMIR->ISA performance result.
+  /// Return nullopt if moduleToObject has not been called or the target format
+  /// is LLVMIR.
+  std::optional<int64_t> getLLVMIRToISAPerfResult();
----------------
joker-eph wrote:

```suggestion
  std::optional<int64_t> getLLVMIRToISATimeInMs();
```

Just a nit: making the name more explicit (I'm assuming the result is in `ms` but please adjust the unit).

(same below)

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