[committed] openmp: Change omp_atv_default value and rename omp_atv_sequential to omp_atv_serialized

[Jakub Jelinek via Gcc-patches]

Hi!

While this is an OpenMP 5.1 change, it is undesirable to let people use 
different
values and then deal with ABI backwards compatibility in a year or two.

Have been waiting with this for a month until the final wording for
omp_atv_default makes it through.

Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk.

2020-07-09  Jakub Jelinek  <ja...@redhat.com>

        * omp.h.in (omp_alloctrait_value_t): Change omp_atv_default from
        2 to -1.  Add omp_atv_serialized and define omp_atv_sequential using
        it.  Remove __omp_alloctrait_value_max__.
        * allocator.c (omp_init_allocator): Handle omp_atv_default for
        omp_atk_alignment and omp_atk_pool_size.

--- libgomp/omp.h.in.jj 2020-05-19 10:09:56.735370056 +0200
+++ libgomp/omp.h.in    2020-06-04 23:24:10.115738893 +0200
@@ -136,12 +136,13 @@ typedef enum omp_alloctrait_key_t
 
 typedef enum omp_alloctrait_value_t
 {
+  omp_atv_default = (__UINTPTR_TYPE__) -1,
   omp_atv_false = 0,
   omp_atv_true = 1,
-  omp_atv_default = 2,
   omp_atv_contended = 3,
   omp_atv_uncontended = 4,
-  omp_atv_sequential = 5,
+  omp_atv_serialized = 5,
+  omp_atv_sequential = omp_atv_serialized,
   omp_atv_private = 6,
   omp_atv_all = 7,
   omp_atv_thread = 8,
@@ -154,8 +155,7 @@ typedef enum omp_alloctrait_value_t
   omp_atv_environment = 15,
   omp_atv_nearest = 16,
   omp_atv_blocked = 17,
-  omp_atv_interleaved = 18,
-  __omp_alloctrait_value_max__ = __UINTPTR_MAX__
+  omp_atv_interleaved = 18
 } omp_alloctrait_value_t;
 
 typedef struct omp_alloctrait_t
--- libgomp/allocator.c.jj      2020-06-02 11:56:59.400537423 +0200
+++ libgomp/allocator.c 2020-06-04 23:29:16.896333916 +0200
@@ -90,6 +90,11 @@ omp_init_allocator (omp_memspace_handle_
          }
        break;
       case omp_atk_alignment:
+        if (traits[i].value == omp_atv_default)
+         {
+           data.alignment = 1;
+           break;
+         }
        if ((traits[i].value & (traits[i].value - 1)) != 0
            || !traits[i].value)
          return omp_null_allocator;
@@ -112,7 +117,10 @@ omp_init_allocator (omp_memspace_handle_
          }
        break;
       case omp_atk_pool_size:
-       data.pool_size = traits[i].value;
+       if (traits[i].value == omp_atv_default)
+         data.pool_size = ~(uintptr_t) 0;
+       else
+         data.pool_size = traits[i].value;
        break;
       case omp_atk_fallback:
        switch (traits[i].value)


        Jakub