This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP][NVPTX]Added/fixed debugging messages, NFC.
ClosedPublic

Authored by ABataev on Dec 27 2018, 7:17 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

ABataev created this revision.Dec 27 2018, 7:17 AM
grokos added inline comments.Dec 27 2018, 8:20 AM
libomptarget/deviceRTLs/nvptx/src/supporti.h
241–242 ↗(On Diff #179552)

Why do we need to cast size to unsigned long long?

ABataev marked an inline comment as done.Dec 27 2018, 8:21 AM
ABataev added inline comments.
libomptarget/deviceRTLs/nvptx/src/supporti.h
241–242 ↗(On Diff #179552)

%zu modifier is not recognized and it leads to the crash at the runtime.

grokos accepted this revision.Dec 27 2018, 8:26 AM

LGTM.

This revision is now accepted and ready to land.Dec 27 2018, 8:26 AM
This revision was automatically updated to reflect the committed changes.