Bugfix of dangling llvm::function_ref

[azecevicTT]

Closes #1753. The description is in the issue. I decided to change llvm::function_ref to std::function. The other option would be to inline lambda directly as an argument to the addToPipeline.

[vprajapati-tt]

Nice catch! Does this help fix the faulty error handling when running the pipelines?

[vprajapati-tt]

LGTM!

[azecevicTT]

Nice catch! Does this help fix the faulty error handling when running the pipelines?

Honestly, I wasn't aware of it, spotted it accidentally while looking at some binding code. Is there some open issue related to it?

[vprajapati-tt]

Nice catch! Does this help fix the faulty error handling when running the pipelines?

Honestly, I wasn't aware of it, spotted it accidentally while looking at some binding code. Is there some open issue related to it?

https://github.com/tenstorrent/tt-forge-fe/blob/64a320069ae631deae0699ca12ced4a1576d7a64/forge/csrc/passes/mlir_passes.cpp#L44 Check this out. This might be helpful to forge-fe.

[azecevicTT]

Nice catch! Does this help fix the faulty error handling when running the pipelines?

Honestly, I wasn't aware of it, spotted it accidentally while looking at some binding code. Is there some open issue related to it?

https://github.com/tenstorrent/tt-forge-fe/blob/64a320069ae631deae0699ca12ced4a1576d7a64/forge/csrc/passes/mlir_passes.cpp#L44 Check this out. This might be helpful to forge-fe.

Thanks for drawing my attention to this! I will look at this, it might help since it's the same error as this one.