Skip to content

Commit 1493f42

Browse files
committed
[OpenACC] Add test changes missed in 7d8da04
Looks like I did a bad job with git and ended up having some failed tests! This should fix those.
1 parent a7a65a8 commit 1493f42

7 files changed

+17
-22
lines changed

clang/test/ParserOpenACC/parse-clauses.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1261,7 +1261,6 @@ void Gang() {
12611261

12621262
}
12631263

1264-
// expected-warning@+6{{OpenACC clause 'nohost' not yet implemented, clause ignored}}
12651264
// expected-warning@+5{{OpenACC construct 'routine' with implicit function not yet implemented, pragma ignored}}
12661265
// expected-error@+4{{OpenACC clause 'seq' may not appear on the same construct as a 'worker' clause on a 'routine' construct}}
12671266
// expected-note@+3{{previous clause is here}}
@@ -1270,7 +1269,6 @@ void Gang() {
12701269
#pragma acc routine worker, vector, seq, nohost
12711270
void bar();
12721271

1273-
// expected-warning@+5{{OpenACC clause 'nohost' not yet implemented, clause ignored}}
12741272
// expected-error@+4{{OpenACC clause 'seq' may not appear on the same construct as a 'worker' clause on a 'routine' construct}}
12751273
// expected-note@+3{{previous clause is here}}
12761274
// expected-error@+2{{OpenACC clause 'vector' may not appear on the same construct as a 'worker' clause on a 'routine' construct}}

clang/test/SemaOpenACC/combined-construct-auto_seq_independent-clauses.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ void uses() {
4747
for(unsigned i = 0; i < 5; ++i);
4848
#pragma acc parallel loop auto vector
4949
for(unsigned i = 0; i < 5; ++i);
50-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
50+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
5151
#pragma acc parallel loop auto nohost
5252
for(unsigned i = 0; i < 5; ++i);
5353
#pragma acc parallel loop auto default(none)
@@ -164,7 +164,7 @@ void uses() {
164164
for(unsigned i = 0; i < 5; ++i);
165165
#pragma acc parallel loop vector auto
166166
for(unsigned i = 0; i < 5; ++i);
167-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
167+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
168168
#pragma acc parallel loop nohost auto
169169
for(unsigned i = 0; i < 5; ++i);
170170
#pragma acc parallel loop default(none) auto
@@ -282,7 +282,7 @@ void uses() {
282282
for(unsigned i = 0; i < 5; ++i);
283283
#pragma acc parallel loop independent vector
284284
for(unsigned i = 0; i < 5; ++i);
285-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
285+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
286286
#pragma acc parallel loop independent nohost
287287
for(unsigned i = 0; i < 5; ++i);
288288
#pragma acc parallel loop independent default(none)
@@ -399,7 +399,7 @@ void uses() {
399399
for(unsigned i = 0; i < 5; ++i);
400400
#pragma acc parallel loop vector independent
401401
for(unsigned i = 0; i < 5; ++i);
402-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
402+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
403403
#pragma acc parallel loop nohost independent
404404
for(unsigned i = 0; i < 5; ++i);
405405
#pragma acc parallel loop default(none) independent
@@ -525,7 +525,7 @@ void uses() {
525525
// expected-error@+1{{OpenACC 'if_present' clause is not valid on 'parallel loop' directive}}
526526
#pragma acc parallel loop seq if_present
527527
for(unsigned i = 0; i < 5; ++i);
528-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
528+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
529529
#pragma acc parallel loop seq nohost
530530
for(unsigned i = 0; i < 5; ++i);
531531
#pragma acc parallel loop seq default(none)
@@ -648,7 +648,7 @@ void uses() {
648648
// expected-error@+1{{OpenACC 'if_present' clause is not valid on 'parallel loop' directive}}
649649
#pragma acc parallel loop if_present seq
650650
for(unsigned i = 0; i < 5; ++i);
651-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
651+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'parallel loop' directive}}
652652
#pragma acc parallel loop nohost seq
653653
for(unsigned i = 0; i < 5; ++i);
654654
#pragma acc parallel loop default(none) seq

clang/test/SemaOpenACC/combined-construct-device_type-clause.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,7 @@ void uses() {
5656
for(int i = 0; i < 5; ++i);
5757
#pragma acc parallel loop device_type(*) worker
5858
for(int i = 0; i < 5; ++i);
59-
// expected-error@+2{{OpenACC clause 'nohost' may not follow a 'device_type' clause in a 'serial loop' construct}}
60-
// expected-note@+1{{previous clause is here}}
59+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'serial loop' directive}}
6160
#pragma acc serial loop device_type(*) nohost
6261
for(int i = 0; i < 5; ++i);
6362
// expected-error@+2{{OpenACC clause 'default' may not follow a 'device_type' clause in a 'kernels loop' construct}}

clang/test/SemaOpenACC/compute-construct-device_type-clause.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,7 @@ void uses() {
6262
// expected-error@+1{{OpenACC 'worker' clause is not valid on 'kernels' directive}}
6363
#pragma acc kernels device_type(*) worker
6464
while(1);
65-
// expected-error@+2{{OpenACC clause 'nohost' may not follow a 'device_type' clause in a 'kernels' construct}}
66-
// expected-note@+1{{previous clause is here}}
65+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'kernels' directive}}
6766
#pragma acc kernels device_type(*) nohost
6867
while(1);
6968
// expected-error@+2{{OpenACC clause 'default' may not follow a 'device_type' clause in a 'kernels' construct}}

clang/test/SemaOpenACC/loop-construct-auto_seq_independent-clauses.c

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ void uses() {
6060
for(unsigned i = 0; i < 5; ++i);
6161
#pragma acc loop auto vector
6262
for(unsigned i = 0; i < 5; ++i);
63-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
63+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
6464
#pragma acc loop auto nohost
6565
for(unsigned i = 0; i < 5; ++i);
6666
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}
@@ -194,7 +194,7 @@ void uses() {
194194
for(unsigned i = 0; i < 5; ++i);
195195
#pragma acc loop vector auto
196196
for(unsigned i = 0; i < 5; ++i);
197-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
197+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
198198
#pragma acc loop nohost auto
199199
for(unsigned i = 0; i < 5; ++i);
200200
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}
@@ -329,7 +329,7 @@ void uses() {
329329
for(unsigned i = 0; i < 5; ++i);
330330
#pragma acc loop independent vector
331331
for(unsigned i = 0; i < 5; ++i);
332-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
332+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
333333
#pragma acc loop independent nohost
334334
for(unsigned i = 0; i < 5; ++i);
335335
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}
@@ -463,7 +463,7 @@ void uses() {
463463
for(unsigned i = 0; i < 5; ++i);
464464
#pragma acc loop vector independent
465465
for(unsigned i = 0; i < 5; ++i);
466-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
466+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
467467
#pragma acc loop nohost independent
468468
for(unsigned i = 0; i < 5; ++i);
469469
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}
@@ -606,7 +606,7 @@ void uses() {
606606
// expected-error@+1{{OpenACC 'if_present' clause is not valid on 'loop' directive}}
607607
#pragma acc loop seq if_present
608608
for(unsigned i = 0; i < 5; ++i);
609-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
609+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
610610
#pragma acc loop seq nohost
611611
for(unsigned i = 0; i < 5; ++i);
612612
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}
@@ -746,7 +746,7 @@ void uses() {
746746
// expected-error@+1{{OpenACC 'if_present' clause is not valid on 'loop' directive}}
747747
#pragma acc loop if_present seq
748748
for(unsigned i = 0; i < 5; ++i);
749-
// expected-warning@+1{{OpenACC clause 'nohost' not yet implemented}}
749+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
750750
#pragma acc loop nohost seq
751751
for(unsigned i = 0; i < 5; ++i);
752752
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}

clang/test/SemaOpenACC/loop-construct-device_type-clause.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,7 @@ void uses() {
5555
for(int i = 0; i < 5; ++i);
5656
#pragma acc loop device_type(*) worker
5757
for(int i = 0; i < 5; ++i);
58-
// expected-error@+2{{OpenACC clause 'nohost' may not follow a 'device_type' clause in a 'loop' construct}}
59-
// expected-note@+1{{previous clause is here}}
58+
// expected-error@+1{{OpenACC 'nohost' clause is not valid on 'loop' directive}}
6059
#pragma acc loop device_type(*) nohost
6160
for(int i = 0; i < 5; ++i);
6261
// expected-error@+1{{OpenACC 'default' clause is not valid on 'loop' directive}}

clang/test/SemaOpenACC/routine-construct-clauses.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,9 @@ void Func();
5656
// expected-error@+2{{OpenACC clause 'gang' may not appear on the same construct as a 'gang' clause on a 'routine' construct}}
5757
// expected-note@+1{{previous clause is here}}
5858
#pragma acc routine(Func) gang gang
59-
// expected-error@+1{{REQUIRED}}
59+
// expected-error@+1{{OpenACC 'routine' construct must have at least one 'gang', 'worker', 'vector' or 'seq' clause}}
6060
#pragma acc routine(Func)
61-
// expected-error@+1{{REQUIRED}}
61+
// expected-error@+1{{OpenACC 'routine' construct must have at least one 'gang', 'worker', 'vector' or 'seq' clause}}
6262
#pragma acc routine(Func) nohost
6363

6464
// only the 'dim' syntax for gang is legal.

0 commit comments

Comments
 (0)