[Openmp-commits] [PATCH] D75581: [libomptarget][amdgcn] Implement get_wtime

[George Rokos via Phabricator via Openmp-commits]

grokos added inline comments.


================
Comment at: openmp/libomptarget/deviceRTLs/amdgcn/src/target_impl.hip:50
+  return ((double)1.0 / 745000000.0) * t;
 }
 
----------------
JonChesterfield wrote:
> I don't have a source for the magic number. It's from before my time. I can offer that rough checks from running applications look ok, and that it has been shipping as part of aomp for ages without user complaints.
At least can you add a comment about what this magic number is (obviously the clock speed in Hz, but it would be nice to have some sort of documentation)? Or (even better) use a macro because this frequency may change in future chips?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75581