Re: [PATCH 01/23] [AARCH64] define word size for lp64 and ilp32

From: Joseph Myers
Date: Tue Jun 28 2016 - 13:48:53 EST


On Tue, 28 Jun 2016, Joseph Myers wrote:

> On Tue, 28 Jun 2016, Yury Norov wrote:
>
> > diff --git a/sysdeps/aarch64/bits/wordsize.h b/sysdeps/aarch64/bits/wordsize.h
>
> See what I said in
> <https://sourceware.org/ml/libc-alpha/2016-06/msg00786.html> about "make
> other bits/wordsize.h files define the macro to 0". This patch would
> break all other architectures (and AArch64 LP64 - you should test build
> and testsuite for that, as well as ILP32, and confirm that there are no
> testsuite regressions for LP64).

(Actually, for LP64, if you can confirm that the installed stripped shared
libraries are byte-for-byte identical before and after the patch series,
then a testsuite run is unnecessary. But you still need to run the
testsuite for ILP32, and justify any failures there, and if the LP64
binaries aren't byte-for-byte identical, you should justify why they
aren't.)

--
Joseph S. Myers
joseph@xxxxxxxxxxxxxxxx