[GitHub] [incubator-mxnet] leezu commented on a change in pull request #18625: Enable Large Tensor Support: Stage 1

[GitBox]

leezu commented on a change in pull request #18625:
URL: https://github.com/apache/incubator-mxnet/pull/18625#discussion_r452385944



##########
File path: ci/docker/runtime_functions.sh
##########
@@ -413,6 +414,7 @@ build_ubuntu_cpu_mkl() {
         -DUSE_TVM_OP=ON \
         -DUSE_MKL_IF_AVAILABLE=ON \
         -DUSE_BLAS=MKL \
+        -DUSE_INT64_TENSOR_SIZE=OFF \

Review comment:
       For OpenBLAS, the respective compile flag to support 64 bit integer is 
`INTERFACE64=1`. We can discuss more if it actually makes sense to use the 
ILP64 data model for blas and lapack or if it's better to throw runtime errors 
or another solution




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org