Fix MIGraphX compilation for v1.19+

[apwojcik]

Additionally, I added the fix for the warning on visibility: The local variable hides the class attribute.

[TedThemistokleous]

Still seeing build failures when I try to compile this. Did this work for you when you tried it?

 59%] Building CXX object CMakeFiles/onnxruntime_framework.dir/onnxruntime/onnxruntime/core/framework/onnxruntime_map_type_info.cc.o
[ 59%] Building CXX object CMakeFiles/onnxruntime_framework.dir/onnxruntime/onnxruntime/core/framework/onnxruntime_optional_type_info.cc.o
/onnxruntime/onnxruntime/core/providers/migraphx/migraphx_execution_provider.cc: In lambda function:
/onnxruntime/onnxruntime/core/providers/migraphx/migraphx_execution_provider.cc:1302:23: error: ‘struct migraphx::api::onnx_options’ has no member named ‘set_external_data_path’
 1302 |           cmp_options.set_external_data_path(model_path_.parent_path().string());

[apwojcik]

Still seeing build failures when I try to compile this. Did this work for you when you tried it?

 59%] Building CXX object CMakeFiles/onnxruntime_framework.dir/onnxruntime/onnxruntime/core/framework/onnxruntime_map_type_info.cc.o
[ 59%] Building CXX object CMakeFiles/onnxruntime_framework.dir/onnxruntime/onnxruntime/core/framework/onnxruntime_optional_type_info.cc.o
/onnxruntime/onnxruntime/core/providers/migraphx/migraphx_execution_provider.cc: In lambda function:
/onnxruntime/onnxruntime/core/providers/migraphx/migraphx_execution_provider.cc:1302:23: error: ‘struct migraphx::api::onnx_options’ has no member named ‘set_external_data_path’
 1302 |           cmp_options.set_external_data_path(model_path_.parent_path().string());

You need recent MIGraphX from the develop branch to build MIGraphX EP. The branch release/rocm-rel-6.2 has that change incorporated, too.

[TedThemistokleous]

Confirmed. Works off develop. Considering this will be bundled with ROCm 6.2 's MIGraphX this is a moot point now.