Skip to content

Commit 1fd6fa5

Browse files
committed
Update diagnostic message and changelogs
1 parent 4dba920 commit 1fd6fa5

9 files changed

+10
-9
lines changed

clang/docs/ReleaseNotes.rst

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -521,7 +521,8 @@ Improvements to Clang's diagnostics
521521
- Fixed a duplicate diagnostic when performing typo correction on function template
522522
calls with explicit template arguments. (#GH139226)
523523

524-
- An error is now emitted when OpenMP ``collapse`` and ``ordered`` clauses has expression larger than 64 bit.
524+
- An error is now emitted when OpenMP ``collapse`` and ``ordered`` clauses have an
525+
argument larger than what can fit within a 64-bit integer.
525526

526527
Improvements to Clang's time-trace
527528
----------------------------------

clang/include/clang/Basic/DiagnosticSemaKinds.td

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11527,7 +11527,7 @@ def note_omp_collapse_ordered_expr : Note<
1152711527
def err_omp_negative_expression_in_clause : Error<
1152811528
"argument to '%0' clause must be a %select{non-negative|strictly positive}1 integer value">;
1152911529
def err_omp_large_expression_in_clause : Error<
11530-
"argument to '%0' clause cannot have more than 64 bits">;
11530+
"argument to '%0' clause requires a value that can be represented by a 64-bit">;
1153111531
def err_omp_not_integral : Error<
1153211532
"expression must have integral or unscoped enumeration "
1153311533
"type, not %0">;

clang/test/OpenMP/for_collapse_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ T tmain(T argc, S **argv) {
4949
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
5050
#pragma omp for collapse (S) // expected-error {{'S' does not refer to a value}}
5151
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
52-
#pragma omp for collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause cannot have more than 64 bits}}
52+
#pragma omp for collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause requires a value that can be represented by a 64-bit}}
5353
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
5454
#if __cplusplus <= 199711L
5555
// expected-error@+4 2 {{integral constant expression}} expected-note@+4 0+{{constant expression}}

clang/test/OpenMP/for_ordered_clause.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ T tmain(T argc, S **argv) {
5353
#pragma omp for ordered(S) // expected-error {{'S' does not refer to a value}}
5454
for (int i = ST; i < N; i++)
5555
argv[0][i] = argv[0][i] - argv[0][i - ST];
56-
#pragma omp for ordered (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'ordered' clause cannot have more than 64 bits}}
56+
#pragma omp for ordered (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'ordered' clause requires a value that can be represented by a 64-bit}}
5757
for (int i = ST; i < N; i++)
5858
argv[0][i] = argv[0][i] - argv[0][i-ST];
5959
#if __cplusplus <= 199711L

clang/test/OpenMP/for_simd_collapse_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ T tmain(T argc, S **argv) {
4343
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4444
#pragma omp for simd collapse (S) // expected-error {{'S' does not refer to a value}}
4545
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
46-
#pragma omp for simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause cannot have more than 64 bits}}
46+
#pragma omp for simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause requires a value that can be represented by a 64-bit}}
4747
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4848
#if __cplusplus <= 199711L
4949
// expected-error@+4 2 {{integral constant expression}} expected-note@+4 0+{{constant expression}}

clang/test/OpenMP/for_simd_loop_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -735,7 +735,7 @@ void test_ordered() {
735735
#pragma omp for simd ordered(1)
736736
for (int i = 0; i < 16; ++i)
737737
;
738-
#pragma omp for simd ordered (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'ordered' clause cannot have more than 64 bits}}
738+
#pragma omp for simd ordered (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'ordered' clause requires a value that can be represented by a 64-bit}}
739739
for (int i = 0; i < 10; i++)
740740
;
741741
}

clang/test/OpenMP/masked_taskloop_collapse_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ T tmain(T argc, S **argv) {
4343
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4444
#pragma omp masked taskloop collapse (S) // expected-error {{'S' does not refer to a value}}
4545
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
46-
#pragma omp masked taskloop collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause cannot have more than 64 bits}}
46+
#pragma omp masked taskloop collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause requires a value that can be represented by a 64-bit}}
4747
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4848
#if __cplusplus <= 199711L
4949
// expected-error@+4 2 {{integral constant expression}} expected-note@+4 0+{{constant expression}}

clang/test/OpenMP/masked_taskloop_simd_collapse_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ T tmain(T argc, S **argv) {
4343
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4444
#pragma omp masked taskloop simd collapse (S) // expected-error {{'S' does not refer to a value}}
4545
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
46-
#pragma omp masked taskloop simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause cannot have more than 64 bits}}
46+
#pragma omp masked taskloop simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause requires a value that can be represented by a 64-bit}}
4747
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4848
#if __cplusplus <= 199711L
4949
// expected-error@+4 2 {{integral constant expression}} expected-note@+4 0+{{constant expression}}

clang/test/OpenMP/simd_collapse_messages.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ T tmain(T argc, S **argv) {
4343
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4444
#pragma omp simd collapse (S) // expected-error {{'S' does not refer to a value}}
4545
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
46-
#pragma omp simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause cannot have more than 64 bits}}
46+
#pragma omp simd collapse (0xFFFFFFFFFFFFFFFF) // expected-error {{argument to 'collapse' clause requires a value that can be represented by a 64-bit}}
4747
for (int i = ST; i < N; i++) argv[0][i] = argv[0][i] - argv[0][i-ST];
4848
#if __cplusplus <= 199711L
4949
// expected-error@+4 2 {{integral constant expression}} expected-note@+4 0+{{constant expression}}

0 commit comments

Comments
 (0)