[GitHub] [incubator-mxnet] access2rohit commented on a change in pull request #17309: adding docs for 64bit C APIs of large tensor and removing not required int64 C APIs

[GitBox <gi...@apache.org>]

access2rohit commented on a change in pull request #17309: adding docs for 64bit C APIs of large tensor and removing not required int64 C APIs 
URL: https://github.com/apache/incubator-mxnet/pull/17309#discussion_r366609591
 
 

 ##########
 File path: include/mxnet/c_api.h
 ##########
 @@ -585,6 +585,8 @@ MXNET_DLL int MXNDArrayCreate(const uint32_t *shape,
 
 /*!
  * \brief create a NDArray with specified shape and data type
+ *  This api is available when MXNet is built with flag
+ *  USE_INT64_TENSOR_SIZE=0 (by default)
 
 Review comment:
   There already is a functionality to throw error messages when usage is of large tensor and mxnet is not built with USE_INT64_TENSOR_SIZE=1. This is a documentation for devs to help them decide which one to choose.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services