Hi! I've noticed that while I have added hopefully sufficient test coverage for the case where one uses simple number or !number as p-interval, I haven't added any coverage for number:len:stride or number:len.
This patch adds that. Tested on x86_64-linux and i686-linux, committed to trunk. 2021-10-15 Jakub Jelinek <ja...@redhat.com> * testsuite/libgomp.c/affinity-1.c (struct places): Change name field type from char [50] to const char *. (places_array): Add a testcase for simplified syntax place followed by length or length and stride. --- libgomp/testsuite/libgomp.c/affinity-1.c.jj 2021-10-15 15:58:51.660135285 +0200 +++ libgomp/testsuite/libgomp.c/affinity-1.c 2021-10-15 17:08:13.638249421 +0200 @@ -48,7 +48,7 @@ struct place }; struct places { - char name[50]; + const char *name; int count; struct place places[8]; } places_array[] = { @@ -63,7 +63,8 @@ struct places { 4, 1 }, { 5, 1 }, { 6, 1 }, { 7, 1 } } }, { "{0,1},{3,2,4},{6,5,!6},{6},{7:2:-1,!6}", 5, { { 0, 2 }, { 2, 3 }, { 5, 1 }, { 6, 1 }, { 7, 1 } } }, - { "1,2,{2,3,!2},3,3,!3,!{5:3:-1,!4,!5},{4},5,!4,!5", 3, + { "1,2,{2,3,!2},3,3,!3,!{5:3:-1,!4,!5},{4},5,!4,!5," + "1:2,!{1},!2,7:3:-2,!{5},!7,!3", 3, { { 1, 1 }, { 2, 1 }, { 3, 1 } } } }; Jakub