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

Fix endianness test on Linux #28

Merged
merged 1 commit into from
Feb 5, 2024

Conversation

ckuethe
Copy link
Contributor

@ckuethe ckuethe commented Feb 4, 2024

Tested on:

  ckuethe@thorium:~$ uname -a
  Linux thorium 6.6.10-76060610-generic #202401051437~1704728131~22.04~24d69e2 SMP PREEMPT_DYNAMIC Mon J x86_64 x86_64 x86_64 GNU/Linux
  ckuethe@thorium:~$ g++ -E -dM - < /dev/null  | grep -E "ORDER|END"
  #define __ORDER_LITTLE_ENDIAN__ 1234
  #define __FLOAT_WORD_ORDER__ __ORDER_LITTLE_ENDIAN__
  #define __ORDER_PDP_ENDIAN__ 3412
  #define __ORDER_BIG_ENDIAN__ 4321
  #define __BYTE_ORDER__ __ORDER_LITTLE_ENDIAN__

Tested on:
```
  ckuethe@thorium:~$ uname -a
  Linux thorium 6.6.10-76060610-generic #202401051437~1704728131~22.04~24d69e2 SMP PREEMPT_DYNAMIC Mon J x86_64 x86_64 x86_64 GNU/Linux
  ckuethe@thorium:~$ g++ -E -dM - < /dev/null  | grep -E "ORDER|END"
  #define __ORDER_LITTLE_ENDIAN__ 1234
  #define __FLOAT_WORD_ORDER__ __ORDER_LITTLE_ENDIAN__
  #define __ORDER_PDP_ENDIAN__ 3412
  #define __ORDER_BIG_ENDIAN__ 4321
  #define __BYTE_ORDER__ __ORDER_LITTLE_ENDIAN__
```
@wcjohns wcjohns merged commit bbeb08e into sandialabs:master Feb 5, 2024
@wcjohns
Copy link
Collaborator

wcjohns commented Feb 5, 2024

Thank you Chris.
Yes, that was definitely a logic inversion on my part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants