On Fri, May 15, 2015 at 1:58 PM, Ramana Radhakrishnan <ramana.radhakrish...@arm.com> wrote: > Thanks for doing this now the patch looks much better and I can apply it ! > Unfortunately there are still problems with it and I'm not sure about your > testing procedures. > >> >> >> mov ip, r0 >> mov r0, r2 >> @@ -1149,12 +1250,15 @@ >> mov r3, ip >> b 6f >> >> -ARM_FUNC_START aeabi_cdcmpeq >> ARM_FUNC_ALIAS aeabi_cdcmple aeabi_cdcmpeq > > > How did this even build ? > > I've cleaned up some lines which were > 80 characters, added a definition > for aeabi_cdcmpeq again and applied the attached after *doing* a full > bootstrap and test run on arm-none-linux-gnueabihf along with a test run on > arm-none-eabi.
Thanks a lot! I guess I should've mentioned I've tested it on arm-rtems411. I'm not sure why it didn't throw me any errors, though, but I'm glad it's finally commited and working. Again, thanks a lot and sorry for the inconvenience. -- Martin Galvan Software Engineer Taller Technologies Argentina San Lorenzo 47, 3rd Floor, Office 5 Córdoba, Argentina Phone: 54 351 4217888 / +54 351 4218211
