On 26/11/19 22:22 +0000, Richard Sandiford wrote:
+ libstd...@gcc.gnu.org
<apin...@marvell.com> writes:
From: Andrew Pinski <apin...@marvell.com>
Hi if we have a aarch64 compiler that has a big-endian
multi-lib, it fails to compile libstdc++ because
simd_fast_mersenne_twister_engine is only defined for little-endian
in ext/random but ext/opt_random.h thinks it is defined always.
OK? Built an aarch64-elf toolchain which has a big-endian multi-lib enabled.
Looks good to me in the sense that I see the same preprocessor
conditions you see. OK if no libstdc++ maintainer tells us it's
the wrong fix in the next 24 hrs.
Looks right.
The #if could even be moved earlier, as the __VEXT macros don't seem
to be used anywhere except that file. If most of the file gets skipped
for big endian, that macro isn't needed either (and it looks like the
__ARM_BIG_ENDIAN version of it is never used at all).
No harm leaving the macro though, so as far as I'm concerned the patch
is fine.
