This is an archive of the discontinued LLVM Phabricator instance.

Differential D45415

[OpenMP] Remove extra warning when we build
ClosedPublic

Authored by guansong on Apr 8 2018, 9:44 AM.

Details

Reviewers
grokos
Commits
rGf679431f916e: [OpenMP] Remove extra warning when we build
rOMP329713: [OpenMP] Remove extra warning when we build
rL329713: [OpenMP] Remove extra warning when we build
Summary

This one line change is to remove this warning message

"warning: integer conversion resulted in a change of sign"

Diff Detail

Event Timeline

guansong created this revision.Apr 8 2018, 9:44 AM
Herald added a subscriber: openmp-commits. · View Herald TranscriptApr 8 2018, 9:44 AM
grokos accepted this revision.Apr 10 2018, 7:42 AM

LGTM

This revision is now accepted and ready to land.Apr 10 2018, 7:42 AM
Closed by commit rL329713: [OpenMP] Remove extra warning when we build (authored by guansong). · Explain WhyApr 10 2018, 8:31 AM
This revision was automatically updated to reflect the committed changes.
Herald added a subscriber: llvm-commits. · View Herald TranscriptApr 10 2018, 8:31 AM

Revision Contents

PathSize
libomptarget/
deviceRTLs/
nvptx/
src/
2 lines

Diff 141549

libomptarget/deviceRTLs/nvptx/src/debug.h

Context not available.
// implement // implement
#if OMPTARGET_NVPTX_DEBUG #if OMPTARGET_NVPTX_DEBUG
#define DON(_flag) ((OMPTARGET_NVPTX_DEBUG) & (_flag)) #define DON(_flag) ((unsigned)(OMPTARGET_NVPTX_DEBUG) & (_flag))
#define PRINT0(_flag, _str) \ #define PRINT0(_flag, _str) \
{ \ { \
Context not available.