Replace enum types in variadic functions by build-in types.

[sfantao]

Summary

When compiling the runtime library with clang we get warnings like:

error: passing an object that undergoes default argument promotion to 'va_start' has undefined behavior [-Werror,-Wvarargs]
    va_start( args, id );
                    ^
note: parameter of type 'kmp_i18n_id_t' (aka 'kmp_i18n_id') is declared here
    kmp_i18n_id_t id,

My understanding is that the va_start macro only gets the promoted type so it won't know what was the exact type of the argument, which can potentially not work for some targets given that the implementation of the the calling convention could not be done properly.

This patch fixes that by using a built-in type in the function signature.

[AndreyChurbanov]

LGTM

runtime/src/kmp_i18n.h

 
 // Two special messages.
 extern kmp_msg_t __kmp_msg_empty;  // Can be used in place where message is required syntactically.
 extern kmp_msg_t __kmp_msg_null;   // Denotes the end of variadic list of arguments.
 
 // Helper functions. Creates messages either from message catalog or from system. Note: these
 // functions allocate memory. You should pass created messages to __kmp_msg() function, it will
 // print messages and destroy them.
-kmp_msg_t  __kmp_msg_format( kmp_i18n_id_t id, ... );
+kmp_msg_t  __kmp_msg_format( unsigned id_arg, ... );
 kmp_msg_t  __kmp_msg_error_code( int code );
 kmp_msg_t  __kmp_msg_error_mesg( char const * mesg );
 
 // Helper macros to make calls shorter.
 #define KMP_MSG( ...  )   __kmp_msg_format( kmp_i18n_msg_ ## __VA_ARGS__ )
 #define KMP_HNT( ...  )   __kmp_msg_format( kmp_i18n_hnt_ ## __VA_ARGS__ )
 #define KMP_SYSERRCODE( code )  __kmp_msg_error_code( code )
 #define KMP_SYSERRMESG( mesg )  __kmp_msg_error_mesg( mesg )

runtime/src/kmp_i18n.c

) {
     __kmp_printf( "%s", buffer->str );
 
 } // __kmp_i18n_dump_catalog
 
 // -------------------------------------------------------------------------------------------------
 
 kmp_msg_t
 __kmp_msg_format(
-    kmp_i18n_id_t id,
+    unsigned id_arg,
     ...
 ) {
 
     kmp_msg_t      msg;
     va_list        args;
     kmp_str_buf_t  buffer;
     __kmp_str_buf_init( & buffer );
 
-    va_start( args, id );
+    va_start( args, id_arg );
+
+    // We use unsigned for the ID argument and explicitly cast it here to the
+    // right enumerator because variadic functions are not compatible with
+    // default promotions.
+    kmp_i18n_id_t id = (kmp_i18n_id_t)id_arg;
+
     #if KMP_OS_UNIX
         // On Linux* OS and OS X*, printf() family functions process parameter numbers, for example:
         // "%2$s %1$s".
         __kmp_str_buf_vprint( & buffer, __kmp_i18n_catgets( id ), args );
     #elif KMP_OS_WINDOWS
         // On Winodws, printf() family functions does not recognize GNU style parameter numbers,
         // so we have to use FormatMessage() instead. It recognizes parameter numbers, e. g.:
         // "%2!s! "%1!s!".

runtime/src/thirdparty/ittnotify/ittnotify_static.c

 ITT_EXTERN_C void _N_(error_handler)(__itt_error_code, va_list args);
 #endif /* ITT_NOTIFY_EXT_REPORT */
 
 #if ITT_PLATFORM==ITT_PLATFORM_WIN
 #pragma warning(push)
 #pragma warning(disable: 4055) /* warning C4055: 'type cast' : from data pointer 'void *' to function pointer 'XXX' */
 #endif /* ITT_PLATFORM==ITT_PLATFORM_WIN */
 
-static void __itt_report_error(__itt_error_code code, ...)
+static void __itt_report_error(unsigned code_arg, ...)
 {
     va_list args;
-    va_start(args, code);
+    va_start(args, code_arg);
+
+    // We use unsigned for the code argument and explicitly cast it here to the
+    // right enumerator because variadic functions are not compatible with
+    // default promotions.
+    __itt_error_code code = (__itt_error_code)code_arg;
+
     if (_N_(_ittapi_global).error_handler != NULL)
     {
         __itt_error_handler_t* handler = (__itt_error_handler_t*)(size_t)_N_(_ittapi_global).error_handler;
         handler(code, args);
     }
 #ifdef ITT_NOTIFY_EXT_REPORT
     _N_(error_handler)(code, args);
 #endif /* ITT_NOTIFY_EXT_REPORT */