At 2011-11-22 18:37:16,"Ramana Radhakrishnan" <ramana.radhakrish...@linaro.org> wrote: > >> > >> 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. > > Xinyu: I seem to have mis-remembered that one of your patches was > turning on Thumb2 for wMMX.
Hi Ramana, I remember I just enable iwmmxt with vfp in the first part of my patch. > > > > > Ramana, in that case, should I add the change you suggested in ARM RTL > > epilogue patch only? > > The comment in Thumb2 epilogues should remain and yes - it should be > added to the ARM RTL epilogue patch only. I'm also ok with that being > in with a #if 0 around it but given it's in the epilogue whoever tries > turning on Thumb2 for iwMMX will surely notice that in the first > testrun :) > > I'll try and find some time later today to apply both the patches and > see where I get to. > > cheers > Ramana > > > > > -- > > > > > >
