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

Reply via email to