On 2014-12-12 05:46, Timothy Arceri wrote:
Signed-off-by: Timothy Arceri <t_arc...@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@desktop bin :) $ ./cc test.c
user@desktop bin :) $ ./a.out
0x590
user@desktop bin :) $


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to