[Openmp-commits] [PATCH] D154456: [Libomptarget] Correctly implement `getWTime` on AMDGPU

[Shilei Tian via Phabricator via Openmp-commits]

tianshilei1992 added inline comments.


================
Comment at: openmp/libomptarget/plugins-nextgen/cuda/src/rtl.cpp:833
+  /// Returns the clock frequency for the given NVPTX device.
+  uint64_t getClockFrequency() const override { return 1000000000; }
+
----------------
This doesn't need to be `1000000000UL`?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154456/new/

https://reviews.llvm.org/D154456