patch for --EmitJNI on windows

[armgong]

this naive fix for the issue 2580 https://github.com/onnx/onnx-mlir/issues/2580
test with add.onnx on windows (jdk 11.0)

[jenkins-droid]

Can one of the admins verify this patch?

[tungld]

@jenkins-droid test this please

[AlexandreEichenberger]

LGTM but I am not an expert. Requested @gongsu832 to double check as he is the one that developed this code.

Thanks for the quick fix.

[AlexandreEichenberger]

please run the clang format on the modified files.

[gongsu832]

For the --EmitJNI option considered to be "working", it should at least pass the JNI backend tests, for which the make targets are:

check-onnx-backend-jni
check-onnx-backend-dynamic-jni
check-onnx-backend-constant-jni
check-onnx-backend-node-jni
check-onnx-backend-model-jni

These tests are currently not run on the Windows CI.

[armgong]

For the --EmitJNI option considered to be "working", it should at least pass the JNI backend tests, for which the make targets are:

check-onnx-backend-jni
check-onnx-backend-dynamic-jni
check-onnx-backend-constant-jni
check-onnx-backend-node-jni
check-onnx-backend-model-jni

These tests are currently not run on the Windows CI.

yeah, try run these tests manually, but the tests themself are no go.

[AlexandreEichenberger]

yeah, try run these tests manually, but the tests themself are no go.

@armgong adding these tests is key to make sure your patch continues to work. Would be great if you could look into adding them permanently once you have a clean build with these additional tests. We have no window servers where we can try this out, so we are reliant on you to perform this task. Your effort is much appreciated.

[armgong]

@AlexandreEichenberger
will try add test for this, but need more time and learn how to add test use python

[hamptonm1]

@armgong Were you still working on this PR? :)

[armgong]

sorry, busy for other things, have no time to mod and finished the test