matteosal commented on issue #20301: URL: https://github.com/apache/incubator-mxnet/issues/20301#issuecomment-877631438
> @matteosal Then why you're disabling it in cmake (DUSE_INT64_TENSOR_SIZE)? I was keeping `USE_INT64_TENSOR_SIZE` disabled because it was causing this problem: https://github.com/apache/incubator-mxnet/issues/19841 But that was on branch 1.x, we are now switching to 2.0 and I've just verified that there is no problem there. So I'd say that ILP64 + USE_INT64_TENSOR_SIZE=ON satisfies all my needs :) Anyway, are `USE_INT64_TENSOR_SIZE` and ILP64 MKL actually expected to go together? If they are not, there should be an explicit consistency check in the cmake script failing with an appropriate message. Building with OpenBLAS [triggers this check](https://github.com/apache/incubator-mxnet/blob/master/cmake/ChooseBlas.cmake#L96), but MKL doesn't. I should also say that building with `-DUSE_INT64_TENSOR_SIZE=OFF` + ILP64 MKL didn't cause the `batch_dot` crash in mxnet 1.x (1.6 in particular). Another question: does the CI test a build linking to ILP64 MKL? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
