On Fri, 16 Sep 2011 14:46:57 +0200 Pierre Vittet <pier...@pvittet.com> wrote:
> Hello, [...] > The patch also remove a preprocessor #if testing if > _STRING_ARCH_unaligned is defined. This symbol is never defined in gcc > and could be only used in CFLAGS. Looking at the code, it does not looks > usefull to define it (and it is only tested on libiberty/md5.c and > libiberty/sha1.c), as we already check the pointer alignement, so > removing it clean the code. I searched on google, and it does not looks > to be used. Does anyone want it or thing that it should not be removed? > > Ok for trunk ? I can't formally approve this patch, but I do hope it will be reviewed and approved soon. See http://gcc.gnu.org/ml/gcc-help/2011-09/msg00126.html and http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00963.html and http://groups.google.com/group/gcc-melt/browse_thread/thread/292c394fea5089c7 Regards. -- Basile STARYNKEVITCH http://starynkevitch.net/Basile/ email: basile<at>starynkevitch<dot>net mobile: +33 6 8501 2359 8, rue de la Faiencerie, 92340 Bourg La Reine, France *** opinions {are only mine, sont seulement les miennes} ***