[OpenMP][GNU] Usage of TASK_TIED constant inside kmp_gsupport.cpp

[vladaindjic]

Summary

The minor code refactorization introduces the TASK_TIED constant inside
kmp_gsupprot.cpp as a replacement for the literal value 1.
The mentioned constant is now used in both kmp_tasking.cpp and
kmp_gsupport.cpp files.

[AndreyChurbanov]

LGTM

[vladaindjic]

@AndreyChurbanov, could you please land this patch for me?

openmp/runtime/src/kmp_gsupport.cpp

void KMP_EXPAND_NAME(KMP_API_NAME_GOMP_TASK)(void (*func)(void *), void *data,
   int gtid = __kmp_entry_gtid();
   kmp_int32 flags = 0;
   kmp_tasking_flags_t *input_flags = (kmp_tasking_flags_t *)&flags;
 
   KA_TRACE(20, ("GOMP_task: T#%d\n", gtid));
 
   // The low-order bit is the "untied" flag
   if (!(gomp_flags & KMP_GOMP_TASK_UNTIED_FLAG)) {
-    input_flags->tiedness = 1;
+    input_flags->tiedness = TASK_TIED;
   }
   // The second low-order bit is the "final" flag
   if (gomp_flags & KMP_GOMP_TASK_FINAL_FLAG) {
     input_flags->final = 1;
   }
   input_flags->native = 1;
   // __kmp_task_alloc() sets up all other flags
 
KA_TRACE(20, (buff, gtid, func, data, copy_func, arg_size, arg_align,
                   gomp_flags, num_tasks, priority, start, end, step));
     __kmp_str_free(&buff);
   }
 #endif
   KMP_ASSERT((size_t)arg_size >= 2 * sizeof(T));
   KMP_ASSERT(arg_align > 0);
   // The low-order bit is the "untied" flag
   if (!(gomp_flags & 1)) {
-    input_flags->tiedness = 1;
+    input_flags->tiedness = TASK_TIED;
   }
   // The second low-order bit is the "final" flag
   if (gomp_flags & 2) {
     input_flags->final = 1;
   }
   // Negative step flag
   if (!up) {
     // If step is flagged as negative, but isn't properly sign extended