On Tue, 2011-11-22 at 01:55 +0000, Xinyu Qi wrote: > At 2011-11-19 07:11:17,"Ramana Radhakrishnan" > <ramana.radhakrish...@linaro.org> wrote: > > On 10 November 2011 18:07, Sameera Deshpande > > <sameera.deshpa...@arm.com> wrote: > > > Please find attached the reworked patch. > > > > OK but for a very small bit . > > > > I'll note that we need to add support for the iwMMXt registers but the > > attached patch (untested) should be what is needed to support the iwMMxt > > registers and this should be something that should be incorporated into your > > ARM RTL epilogues patch as well. My understanding is that this is all you > > should need to do as there is a wldrd as a part of *iwmmx*_movdi instruction > > and therefore this should just work in this form. > > > > Xinyu can you help in testing this patch once this is applied along with > > your > > other patches to turn on Thumb2 for iwmmxt ? > > > > cheers > > Ramana > > Hi, Ramana, > > Though I don't fully understand this patch, I think it is unnecessary to > consider turn on Thumb2 for iwmmxt, > because there is no chip that supports WMMX instructions with thumb2 mode. > > Thanks, > Xinyu >
Ramana, in that case, should I add the change you suggested in ARM RTL epilogue patch only? --
