Skip to content

Commit

Permalink
Add support for aarch64_be, or1k and microblazebe.
Browse files Browse the repository at this point in the history 
Fixes #73.
  • Loading branch information
@floitsch
floitsch committed Sep 8, 2018
1 parent 4e8b3b5 commit 768a445
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions double-conversion/utils.h
View file
Expand Up @@ -76,8 +76,9 @@ inline void abort_noreturn() { abort(); }
defined(__sparc__) || defined(__sparc) || defined(__s390__) || \
defined(__SH4__) || defined(__alpha__) || \
defined(_MIPS_ARCH_MIPS32R2) || \
defined(__AARCH64EL__) || defined(__aarch64__) || \
defined(__riscv)
defined(__AARCH64EL__) || defined(__aarch64__) || defined(__AARCH64EB__) || \
defined(__riscv) || \
defined(__or1k__)
#define DOUBLE_CONVERSION_CORRECT_DOUBLE_OPERATIONS 1
#elif defined(__mc68000__) || \
defined(__pnacl__) || defined(__native_client__)
Expand Down

2 comments on commit 768a445

@pseiderer
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for merging the suggested changes, but your seem to have missed the
'defined(microblaze)' part (despite mentioning it in the commit subject ;-))...

Detected while fixing a buildroot compile failure of another double-conversion using package (this time icu, see [1] for details)...

[1] http://lists.busybox.net/pipermail/buildroot/2019-October/261553.html

@floitsch
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Argh.
Sorry about that. I missed the line in the original patch.
Should be fixed now. (5cc233e)

Please sign in to comment.