[Mesa-dev] [RFC 6/8] mesa: remove support for GCC older than 4.1.0

kallisti5 kallisti5 at ssl.unixzen.com
Fri Dec 12 05:01:18 PST 2014


On 2014-12-12 05:46, Timothy Arceri wrote:
> Signed-off-by: Timothy Arceri <t_arceri at yahoo.com.au>
> ---
>  src/mesa/main/compiler.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mesa/main/compiler.h b/src/mesa/main/compiler.h
> index 34671dc..cdc843d 100644
> --- a/src/mesa/main/compiler.h
> +++ b/src/mesa/main/compiler.h
> @@ -122,7 +122,7 @@ extern "C" {
>   * inline a static function that we later use in an alias. - ajax
>   */
>  #ifndef PUBLIC
> -#  if (defined(__GNUC__) && __GNUC__ >= 4) || (defined(__SUNPRO_C) &&
> (__SUNPRO_C >= 0x590))
> +#  if defined(__GNUC__) || (defined(__SUNPRO_C) && (__SUNPRO_C >= 
> 0x590))
>  #    define PUBLIC __attribute__((visibility("default")))
>  #    define USED __attribute__((used))
>  #  else

Why not also drop the SUNPRO_C version check?

#  if defined(__GNUC__) || defined(__SUNPRO_C)

I downloaded SunStudio 12 (circa 2007) and __SUNPRO_C is set to 0x590.
If we drop support for GCC 4.1 (2007), might as well drop older sun 
compilers.

  -- Alex



#include <stdio.h>

int
main()
{
     printf("0x%X\n", __SUNPRO_C);
     return 0;
}


user at desktop bin :) $ ./cc test.c
user at desktop bin :) $ ./a.out
0x590
user at desktop bin :) $




More information about the mesa-dev mailing list