[Mlir-commits] [mlir] [mlir][gpu] Expose some utility functions from `gpu-to-binary` infra (PR #172205)

[Fabian Mora]

================
@@ -98,11 +105,6 @@ class ModuleToObject {
   /// Optimize the module.
   virtual LogicalResult optimizeModule(llvm::Module &module, int optL);
 
-  /// Utility function for translating to ISA, returns `std::nullopt` on
-  /// failure.
-  static std::optional<std::string>
-  translateToISA(llvm::Module &llvmModule, llvm::TargetMachine &targetMachine);
-
----------------
fabianmcg wrote:

What is the purpose of moving this out of the class? It's already static and the class is non-template. To me it makes sense to be scoped under the class.

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