[Openmp-commits] [PATCH] D85353: [libomptarget][NFC] Replace `%ld` with PRId64 for data of type int64_t.

George Rokos via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Wed Aug 5 13:17:34 PDT 2020


grokos created this revision.
grokos added reviewers: Hahnfeld, jdoerfert, ABataev.
grokos added a project: OpenMP.
grokos requested review of this revision.
Herald added a subscriber: sstefan1.

The standard way of printing `int64_t` data is via the PRId64 macro, `ld` is for `long int` and int64_t is not guaranteed to be typedef'ed as `long int` on all platforms.
E.g. on Windows we get mismatch warnings.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D85353

Files:
  openmp/libomptarget/plugins/cuda/src/rtl.cpp
  openmp/libomptarget/src/device.cpp
  openmp/libomptarget/src/omptarget.cpp
  openmp/libomptarget/src/rtl.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D85353.283362.patch
Type: text/x-patch
Size: 5901 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20200805/01145288/attachment-0001.bin>


More information about the Openmp-commits mailing list