This is an archive of the discontinued LLVM Phabricator instance.

Differential D106777

[libomptarget][nfc] Squash unused variable warning
ClosedPublic

Authored by JonChesterfield on Jul 25 2021, 4:31 PM.

Details

Reviewers
jdenny
jdoerfert
Commits
rG93fe84d32fea: [libomptarget][nfc] Squash unused variable warning
Summary

Suppress only current warning on openmp-clang-x86_64-linux-debian

Diff Detail

Event Timeline

JonChesterfield created this revision.Jul 25 2021, 4:31 PM
Herald added a subscriber: pengfei. · View Herald TranscriptJul 25 2021, 4:31 PM
JonChesterfield requested review of this revision.Jul 25 2021, 4:31 PM
Herald added a reviewer: jdoerfert. · View Herald TranscriptJul 25 2021, 4:31 PM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: openmp-commits, sstefan1. · View Herald Transcript
jdoerfert accepted this revision.Jul 25 2021, 4:52 PM

Just commit things like this please.

This revision is now accepted and ready to land.Jul 25 2021, 4:52 PM
Harbormaster completed remote builds in B116112: Diff 361557.Jul 25 2021, 5:04 PM
Closed by commit rG93fe84d32fea: [libomptarget][nfc] Squash unused variable warning (authored by JonChesterfield). · Explain WhyJul 26 2021, 1:54 AM
This revision was automatically updated to reflect the committed changes.
JonChesterfield added a commit: rG93fe84d32fea: [libomptarget][nfc] Squash unused variable warning.

Revision Contents

PathSize
openmp/
libomptarget/
src/
1 line

Diff 361595

openmp/libomptarget/src/device.cpp

Show First 20 Lines • Show All 84 Lines • ▼ Show 20 Lines const HostDataToTargetTy &newEntry =
/*TgtPtrBegin=*/(uintptr_t)TgtPtrBegin, /*Name=*/nullptr, /*TgtPtrBegin=*/(uintptr_t)TgtPtrBegin, /*Name=*/nullptr,
/*IsRefCountINF=*/true) /*IsRefCountINF=*/true)
.first; .first;
DP("Creating new map entry: HstBase=" DPxMOD ", HstBegin=" DPxMOD DP("Creating new map entry: HstBase=" DPxMOD ", HstBegin=" DPxMOD
", HstEnd=" DPxMOD ", TgtBegin=" DPxMOD ", RefCount=%s\n", ", HstEnd=" DPxMOD ", TgtBegin=" DPxMOD ", RefCount=%s\n",
DPxPTR(newEntry.HstPtrBase), DPxPTR(newEntry.HstPtrBegin), DPxPTR(newEntry.HstPtrBase), DPxPTR(newEntry.HstPtrBegin),
DPxPTR(newEntry.HstPtrEnd), DPxPTR(newEntry.TgtPtrBegin), DPxPTR(newEntry.HstPtrEnd), DPxPTR(newEntry.TgtPtrBegin),
newEntry.refCountToStr().c_str()); newEntry.refCountToStr().c_str());
(void)newEntry;
DataMapMtx.unlock(); DataMapMtx.unlock();
return OFFLOAD_SUCCESS; return OFFLOAD_SUCCESS;
} }
int DeviceTy::disassociatePtr(void *HstPtrBegin) { int DeviceTy::disassociatePtr(void *HstPtrBegin) {
DataMapMtx.lock(); DataMapMtx.lock();
▲ Show 20 Lines • Show All 476 Lines • Show Last 20 Lines