Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add condition for sparcv8plus to knows instruction sets #2491

Closed
wants to merge 1 commit into from

Conversation

dago
Copy link
Contributor

@dago dago commented Apr 13, 2018

With specific compiler options selecting the arch like -xarch=sparc on newer compilers like Oracle Studio 12.4 there is no definition of __sparcv8 but __sparcv8plus which means the V9 ISA, but limited to the 32–bit subset defined by the V8plus ISA specification, without the Visual Instruction Set (VIS), and without other implementation-specific ISA extensions. So it should be the same as __sparcv8.

@dago
Copy link
Contributor Author

dago commented Apr 13, 2018

Just for reference here is the current list of predefined macros with the compiler settings which triggered the error. Please note that __sparcv8 is missing but __sparcv8plus is present.

dam@unstable10s [unstable10s]:/home/dam > /opt/solarisstudio12.4/bin/cc -xarch=sparc -std=c99 -I/opt/csw/include -xO3 -m32  -DSSL_EXPERIMENTAL -DSSL_ENGINE -DSOLARIS2=10 -D_POSIX_PTHREAD_SEMANTICS -D_REENTRANT -D_LARGEFILE64_SOURCE -E -xdumpmacros /dev/null
#define __LINE__ 
#define __FILE__ 
#define __STDC__ 0
#define __STDC_VERSION__ 199901L
#define __DATE__ "Apr 13 2018"
#define __TIME__ "11:45:30"
#define __STDC_IEC_559__ 1
#define __STDC_IEC_559_COMPLEX__ 1
#define __STDC_HOSTED__ 1
#define __STDC_ANALYZABLE__ 0
#define __STDC_NO_ATOMICS__ 0
#define __STDC_NO_THREADS__ 0
#define __has_attribute (attr) ___has_attribute___ ( attr )
#define SSL_EXPERIMENTAL 1
#define SSL_ENGINE 1
#define SOLARIS2 10
#define _POSIX_PTHREAD_SEMANTICS 1
#define _REENTRANT 1
#define _LARGEFILE64_SOURCE 1
#define __SunOS_5_10 1
#define __SUNPRO_C 0x5130
#define __unix 1
#define __SVR4 1
#define __sun 1
#define __SunOS 1
#define __sparcv8plus 1
#define __sparc 1
#define __BUILTIN_VA_ARG_INCR 1
#define __C99FEATURES__ 1
#define __PRAGMA_REDEFINE_EXTNAME 1
#define unix 1
#define sun 1
#define sparc 1
#define __RESTRICT 1
#define __FLT_EVAL_METHOD__ 0
#define __SUN_PREFETCH 1
# 1 "/dev/null"
#ident "acomp: Sun C 5.13 SunOS_sparc Patch 151632-06 2016/07/19"

@jay jay closed this in 631f64c Apr 15, 2018
@jay
Copy link
Member

jay commented Apr 15, 2018

Thanks

@dago dago deleted the sparcv8plus branch April 16, 2018 09:11
@lock lock bot locked as resolved and limited conversation to collaborators Jul 15, 2018
@dago dago restored the sparcv8plus branch July 4, 2019 07:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants