This is an archive of the discontinued LLVM Phabricator instance.

Differential D86758

[OpenMP] Fixed wrong test command in the test private_mapping.c
ClosedPublic

Authored by tianshilei1992 on Aug 27 2020, 7:54 PM.

Details

Reviewers
ABataev
jdoerfert
Commits
rG46e0ced762ce: [OpenMP] Fixed wrong test command in the test private_mapping.c
Summary

The test command in private_mapping.c was set to expect failure by mistake. It is fixed in this patch.

Diff Detail

Unit TestsFailed

TimeTest
420 mslinux > libomp.ompt/tasks::dependences.c

Event Timeline

tianshilei1992 created this revision.Aug 27 2020, 7:54 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 27 2020, 7:54 PM
Herald added subscribers: openmp-commits, guansong, yaxunl. · View Herald Transcript
tianshilei1992 requested review of this revision.Aug 27 2020, 7:54 PM
Herald added a reviewer: jdoerfert. · View Herald TranscriptAug 27 2020, 7:54 PM
Herald added a subscriber: sstefan1. · View Herald Transcript
Harbormaster completed remote builds in B69873: Diff 288513.Aug 27 2020, 8:18 PM
ABataev added a comment.Aug 28 2020, 3:14 AM

compilexx is for c++ tests, IIRC, try compile instead

tianshilei1992 updated this revision to Diff 288614.Aug 28 2020, 8:13 AM

Updated based on comments

ABataev accepted this revision.Aug 28 2020, 8:14 AM

LG

This revision is now accepted and ready to land.Aug 28 2020, 8:14 AM
Harbormaster completed remote builds in B69921: Diff 288614.Aug 28 2020, 8:54 AM
This revision was landed with ongoing or failed builds.Aug 28 2020, 9:19 AM
Closed by commit rG46e0ced762ce: [OpenMP] Fixed wrong test command in the test private_mapping.c (authored by tianshilei1992). · Explain Why
This revision was automatically updated to reflect the committed changes.
tianshilei1992 added a commit: rG46e0ced762ce: [OpenMP] Fixed wrong test command in the test private_mapping.c.

Revision Contents

PathSize
openmp/
libomptarget/
test/
mapping/
24 lines

Diff 288614

openmp/libomptarget/test/mapping/private_mapping.c

// RUN: %libomptarget-compile-aarch64-unknown-linux-gnu // RUN: %libomptarget-compile-run-and-check-aarch64-unknown-linux-gnu
// RUN: %libomptarget-run-fail-aarch64-unknown-linux-gnu 2>&1 \ // RUN: %libomptarget-compile-run-and-check-powerpc64-ibm-linux-gnu
// RUN: | %fcheck-aarch64-unknown-linux-gnu // RUN: %libomptarget-compile-run-and-check-powerpc64le-ibm-linux-gnu
// RUN: %libomptarget-compile-run-and-check-x86_64-pc-linux-gnu
// RUN: %libomptarget-compile-powerpc64-ibm-linux-gnu // RUN: %libomptarget-compile-run-and-check-nvptx64-nvidia-cuda
// RUN: %libomptarget-run-fail-powerpc64-ibm-linux-gnu 2>&1 \
// RUN: | %fcheck-powerpc64-ibm-linux-gnu
// RUN: %libomptarget-compile-powerpc64le-ibm-linux-gnu
// RUN: %libomptarget-run-fail-powerpc64le-ibm-linux-gnu 2>&1 \
// RUN: | %fcheck-powerpc64le-ibm-linux-gnu
// RUN: %libomptarget-compile-x86_64-pc-linux-gnu
// RUN: %libomptarget-run-fail-x86_64-pc-linux-gnu 2>&1 \
// RUN: | %fcheck-x86_64-pc-linux-gnu
// RUN: %libomptarget-compile-nvptx64-nvidia-cuda
// RUN: %libomptarget-run-fail-nvptx64-nvidia-cuda 2>&1 \
// RUN: | %fcheck-nvptx64-nvidia-cuda
#include <assert.h> #include <assert.h>
#include <stdio.h> #include <stdio.h>
int main() { int main() {
int data1[3] = {1}, data2[3] = {2}, data3[3] = {3}; int data1[3] = {1}, data2[3] = {2}, data3[3] = {3};
int sum[16] = {0}; int sum[16] = {0};
#pragma omp target teams distribute parallel for map(tofrom \ #pragma omp target teams distribute parallel for map(tofrom \
Show All 20 Lines