Should I pass values or references of mlir types as function arguments

[qliu93]

Hi,

Are Op and Value lightweight references? Which of the following is suggested?

// style A
void myfunc(mlir::SomeOp& op, mlir::Value& value);

// style B
void myfunc(mlir::SomeOp op, mlir::Value value);

Thanks in advance

[stellaraccident]

@qliu93

// style B
void myfunc(mlir::SomeOp op, mlir::Value 

Style B - the Op and Value types are effectively thin c++ wrappers around pointers meant to be passed by value.