remove --use_openmp in build.sh

[fumihwh]

Remove --use_openmp in build.sh to fix build failed problem on Mac.
#3313

[Craigacp]

If you've got homebrew running on your Mac you can install libomp and then this patch should make it build - https://gist.github.com/Craigacp/66be36737e21df87eee4c0197e76862a.

[fumihwh]

If you've got homebrew running on your Mac you can install libomp and then this patch should make it build - https://gist.github.com/Craigacp/66be36737e21df87eee4c0197e76862a.

@Craigacp Thanks for reply.
As --use_openmp can be passed to build.sh as an argument, we should not put it into build.sh as default and can not be set to false.

[wschin]

openmp is an important part for computation. I feel if Mac build fails because of it, we need to fix Mac build instead of getting rid of a part of ONNXRuntime. Any comments?

[snnn]

It's not for removing openmp from our build pipelines. As far as I know, none of these are using this script. But maybe I'm wrong, please double check. Maybe there is a less common pipeline I don't know.

[yufenglee]

If you've got homebrew running on your Mac you can install libomp and then this patch should make it build - https://gist.github.com/Craigacp/66be36737e21df87eee4c0197e76862a.

@Craigacp Thanks for reply.
As --use_openmp can be passed to build.sh as an argument, we should not put it into build.sh as default and can not be set to false.

Yes, that's right. In addition, build.bat doesn't set use_openmp by default. It's really confusing.
@fumihwh, we have dock files using build.sh. Please update them by adding --use_openmp to keep their build options.

[snnn]

Hi @fumihwh, could you please update your branch with the latest master? I couldn't schedule the some CI builds for you.

[snnn]

Thank you! @fumihwh,

[fumihwh]

@snnn Sorry for the late reply. Glad to see the change is merged.