FrozenGene commented on a change in pull request #7020:
URL: https://github.com/apache/tvm/pull/7020#discussion_r534719806
##########
File path: include/tvm/auto_scheduler/search_task.h
##########
@@ -44,17 +44,16 @@ class HardwareParamsNode : public Object {
int cache_line_bytes;
// GPU related parameters got from device query API
-
- /*! \brief The max shared memory per block. */
- int max_shared_memory_per_block{INT32_MAX};
- /*! \brief The max register memory per block. */
- int max_registers_per_block{INT32_MAX};
- /*! \brief The max threads per block. */
- int max_threads_per_block{INT32_MAX};
+ /*! \brief The max shared memory per block in bytes. */
+ int max_shared_memory_per_block;
+ /*! \brief The max number of register per block. */
+ int max_registers_per_block;
+ /*! \brief The max number of threads per block. */
+ int max_threads_per_block;
Review comment:
How about exposing them under the `tvm/_ffi/runtime_ctypes.py` like
```python
@property
def max_thread_dimensions(self):
```
This will bring convenience when we want to get these values in our
`HardwareParams` like (mali target)
----------------------------------------------------------------
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
