[Openmp-commits] [PATCH] D109728: [OpenMP][DeviceRTL] Add the support for printf in a freestanding way

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

I think clang will lower printf for nvptx targets to vprintf. Can you check that. Also check what it emits for amdgcn targets.



================
Comment at: openmp/libomptarget/DeviceRTL/src/Debug.cpp:37
+extern "C" {
+inline __attribute__((always_inline)) int printf(const char *format, ...) {
+  return 0;
----------------
no need for these annotations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109728