[tvm] branch main updated: [OpenCL] Use size_t instead of int64_t for OpenCL timer count (#12328)

[mousius]

This is an automated email from the ASF dual-hosted git repository.

mousius pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/tvm.git


The following commit(s) were added to refs/heads/main by this push:
     new 9b86009553 [OpenCL] Use size_t instead of int64_t for OpenCL timer 
count (#12328)
9b86009553 is described below

commit 9b860095532d2a01525a449fac2bdfc0813bf4cc
Author: Eric Lunderberg <lunderb...@users.noreply.github.com>
AuthorDate: Mon Aug 8 06:04:11 2022 -0500

    [OpenCL] Use size_t instead of int64_t for OpenCL timer count (#12328)
    
    Resolves a few gcc warnings for comparing signed and unsigned
    integers.
---
 src/runtime/opencl/opencl_common.h      | 8 ++++----
 src/runtime/opencl/opencl_device_api.cc | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/runtime/opencl/opencl_common.h 
b/src/runtime/opencl/opencl_common.h
index e940d980c4..f16e1e936d 100644
--- a/src/runtime/opencl/opencl_common.h
+++ b/src/runtime/opencl/opencl_common.h
@@ -456,11 +456,11 @@ class OpenCLTimerNode : public TimerNode {
   virtual void Stop() {
     std::vector<cl_event> evt_queue = 
cl::OpenCLWorkspace::Global()->GetEventQueue(dev_);
     cl_ulong start, end;
-    int64_t start_idx = event_start_idxs[count_timer_execs - 1];
+    size_t start_idx = event_start_idxs[count_timer_execs - 1];
 
     if (cl::OpenCLWorkspace::Global()->GetEventQueue(dev_).size() > 0) {
       OPENCL_CALL(clWaitForEvents(1, 
&(cl::OpenCLWorkspace::Global()->GetEventQueue(dev_).back())));
-      for (int i = start_idx; i < evt_queue.size(); ++i) {
+      for (size_t i = start_idx; i < evt_queue.size(); ++i) {
         auto& kevt = evt_queue[i];
         OPENCL_CALL(clGetEventProfilingInfo(kevt, CL_PROFILING_COMMAND_START, 
sizeof(cl_ulong),
                                             &start, nullptr));
@@ -488,8 +488,8 @@ class OpenCLTimerNode : public TimerNode {
   explicit OpenCLTimerNode(Device dev) : dev_(dev) {}
 
   static constexpr const char* _type_key = "OpenCLTimerNode";
-  static int64_t count_timer_execs;
-  static std::vector<int64_t> event_start_idxs;
+  static size_t count_timer_execs;
+  static std::vector<size_t> event_start_idxs;
   TVM_DECLARE_FINAL_OBJECT_INFO(OpenCLTimerNode, TimerNode);
 
  private:
diff --git a/src/runtime/opencl/opencl_device_api.cc 
b/src/runtime/opencl/opencl_device_api.cc
index 3f6649fd9f..cea0acc07c 100644
--- a/src/runtime/opencl/opencl_device_api.cc
+++ b/src/runtime/opencl/opencl_device_api.cc
@@ -492,7 +492,7 @@ 
TVM_REGISTER_GLOBAL("profiling.timer.opencl").set_body_typed([](Device dev) {
 });
 
 }  // namespace cl
-int64_t OpenCLTimerNode::count_timer_execs = 0;
-std::vector<int64_t> OpenCLTimerNode::event_start_idxs;
+size_t OpenCLTimerNode::count_timer_execs = 0;
+std::vector<size_t> OpenCLTimerNode::event_start_idxs;
 }  // namespace runtime
 }  // namespace tvm