On 10 December 2014 at 09:51, Alan Hayward <alan.hayw...@arm.com> wrote:
>>>>This is a new version of my BE patch from a few weeks ago. >>>>This is part 1 and covers rtlanal.c. The second part will be aarch64 >>>>specific. >>>> >>>>When combined with the second patch, It fixes up movoi/ci/xi for Big >>>>Endian, so that we end up with the lab of a big-endian integer to be in >>>>the low byte of the highest-numbered register. >>>> >>>>This will apply cleanly by itself and no regressions were seen when >>>>testing aarch64 and x86_64 on make check. Hi Alan, I'm not a maintainer in this area of the compiler... however I would suggest that the relevant maintainers of the middle end rtlanal.c etc are more likely to pick up this review if the text of the email focussed on how rtlanal.c is broken and why the proposed patch is the right way to go. You should also CC one of the maintainers... Cheers /Marcus
