0a2b6f6c1c
While doing lowering, I discovered that the restriction onthe allowed modifiers for 'copyout' didn't make sense! After discussion on the OpenACC standards mailing list I discovered that this was a copy/paste error during standardization that they intend to fix, and really meant for copyout to allow alwaysout instead of alwaysin. When implementing, I blindly followed the standard :) This patch corrects the implementation to do what was meant.
97 lines
4.5 KiB
C
97 lines
4.5 KiB
C
// RUN: %clang_cc1 %s -fopenacc -verify
|
|
|
|
typedef struct IsComplete {
|
|
struct S { int A; } CompositeMember;
|
|
int ScalarMember;
|
|
float ArrayMember[5];
|
|
void *PointerMember;
|
|
} Complete;
|
|
void uses(int IntParam, short *PointerParam, float ArrayParam[5], Complete CompositeParam) {
|
|
int LocalInt;
|
|
short *LocalPointer;
|
|
float LocalArray[5];
|
|
Complete LocalComposite;
|
|
// Check Appertainment:
|
|
#pragma acc parallel loop copyout(LocalInt)
|
|
for(int i = 0; i < 5; ++i);
|
|
#pragma acc serial loop copyout(LocalInt)
|
|
for(int i = 0; i < 5; ++i);
|
|
#pragma acc kernels loop copyout(LocalInt)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-warning@+1{{OpenACC clause name 'pcopyout' is a deprecated clause name and is now an alias for 'copyout'}}
|
|
#pragma acc parallel loop pcopyout(LocalInt)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-warning@+1{{OpenACC clause name 'present_or_copyout' is a deprecated clause name and is now an alias for 'copyout'}}
|
|
#pragma acc parallel loop present_or_copyout(LocalInt)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// Valid cases:
|
|
#pragma acc parallel loop copyout(LocalInt, LocalPointer, LocalArray)
|
|
for(int i = 0; i < 5; ++i);
|
|
#pragma acc parallel loop copyout(LocalArray[2:1])
|
|
for(int i = 0; i < 5; ++i);
|
|
#pragma acc parallel loop copyout(zero:LocalArray[2:1])
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
#pragma acc parallel loop copyout(LocalComposite.ScalarMember, LocalComposite.ScalarMember)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+1{{OpenACC variable is not a valid variable name, sub-array, array element, member of a composite variable, or composite variable member}}
|
|
#pragma acc parallel loop copyout(1 + IntParam)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+1{{OpenACC variable is not a valid variable name, sub-array, array element, member of a composite variable, or composite variable member}}
|
|
#pragma acc parallel loop copyout(+IntParam)
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+1{{OpenACC sub-array length is unspecified and cannot be inferred because the subscripted value is not an array}}
|
|
#pragma acc parallel loop copyout(PointerParam[2:])
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+1{{OpenACC sub-array specified range [2:5] would be out of the range of the subscripted array size of 5}}
|
|
#pragma acc parallel loop copyout(ArrayParam[2:5])
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+2{{OpenACC sub-array specified range [2:5] would be out of the range of the subscripted array size of 5}}
|
|
// expected-error@+1{{OpenACC variable is not a valid variable name, sub-array, array element, member of a composite variable, or composite variable member}}
|
|
#pragma acc parallel loop copyout((float*)ArrayParam[2:5])
|
|
for(int i = 0; i < 5; ++i);
|
|
// expected-error@+1{{OpenACC variable is not a valid variable name, sub-array, array element, member of a composite variable, or composite variable member}}
|
|
#pragma acc parallel loop copyout((float)ArrayParam[2])
|
|
for(int i = 0; i < 5; ++i);
|
|
// expected-error@+2{{unknown modifier 'invalid' in OpenACC modifier-list on 'copyout' clause}}
|
|
// expected-error@+1{{OpenACC variable is not a valid variable name, sub-array, array element, member of a composite variable, or composite variable member}}
|
|
#pragma acc parallel loop copyout(invalid:(float)ArrayParam[2])
|
|
for(int i = 0; i < 5; ++i);
|
|
|
|
// expected-error@+1{{OpenACC 'copyout' clause is not valid on 'loop' directive}}
|
|
#pragma acc loop copyout(LocalInt)
|
|
for(int i = 0; i < 6;++i);
|
|
// expected-error@+1{{OpenACC 'pcopyout' clause is not valid on 'loop' directive}}
|
|
#pragma acc loop pcopyout(LocalInt)
|
|
for(int i = 0; i < 6;++i);
|
|
// expected-error@+1{{OpenACC 'present_or_copyout' clause is not valid on 'loop' directive}}
|
|
#pragma acc loop present_or_copyout(LocalInt)
|
|
for(int i = 0; i < 6;++i);
|
|
}
|
|
void ModList() {
|
|
int V1;
|
|
// expected-error@+2{{OpenACC 'alwaysin' modifier not valid on 'copyout' clause}}
|
|
// expected-error@+1{{OpenACC 'readonly' modifier not valid on 'copyout' clause}}
|
|
#pragma acc parallel loop copyout(always, alwaysin, alwaysout, zero, readonly: V1)
|
|
for(int i = 0; i < 6;++i);
|
|
// expected-error@+1{{OpenACC 'alwaysin' modifier not valid on 'copyout' clause}}
|
|
#pragma acc serial loop copyout(alwaysin: V1)
|
|
for(int i = 0; i < 6;++i);
|
|
// expected-error@+1{{OpenACC 'readonly' modifier not valid on 'copyout' clause}}
|
|
#pragma acc kernels loop copyout(readonly: V1)
|
|
for(int i = 0; i < 6;++i);
|
|
#pragma acc parallel loop copyout(capture:V1)
|
|
for(int i = 5; i < 10;++i);
|
|
#pragma acc parallel loop copyout(always, alwaysout, zero, capture: V1)
|
|
for(int i = 5; i < 10;++i);
|
|
}
|
|
|