[mlir] Remove unnecessary copying of pass options

[csigg]

Summary

I missed a comment in D98279 that you don't need to copy pass options.

[csigg]

I'm wondering though, wouldn't it be cleaner to implement the copy c'tor for Pass::Option so that passes (like SerializeToBlobPass here) can use the default copy c'tor?

mlir/lib/Dialect/GPU/Transforms/SerializeToBlob.cpp

 using namespace mlir;
 
 std::string gpu::getDefaultGpuBinaryAnnotation() { return "gpu.binary"; }
 
 gpu::SerializeToBlobPass::SerializeToBlobPass(TypeID passID)
     : OperationPass<gpu::GPUModuleOp>(passID) {}
 
 gpu::SerializeToBlobPass::SerializeToBlobPass(const SerializeToBlobPass &other)
-    : OperationPass<gpu::GPUModuleOp>(other) {
-  // Pass::Option has no copy constructor, copy them manually.
-  triple = other.triple;
-  chip = other.chip;
-  features = other.features;
-  gpuBinaryAnnotation = other.gpuBinaryAnnotation;
-}
+    : OperationPass<gpu::GPUModuleOp>(other) {}
 
 static std::string translateToISA(llvm::Module &llvmModule,
                                   llvm::TargetMachine &targetMachine) {
   llvmModule.setDataLayout(targetMachine.createDataLayout());
 
   std::string targetISA;
   llvm::raw_string_ostream stream(targetISA);
   llvm::buffer_ostream pstream(stream);