Hi Jim, On 10/06/16 23:48, Jim Wilson wrote:
This adds a tuning structure for qdf24xx. This was tested with an aarch64-linux bootstrap and a make check, with no regressions. I also tested it with an x86_64-linux C make check to verify that I didn't break the testsuite for non aarch64 targets.
As this also changes code in the arm backend it also needs a bootstrap and test on an arm target (arm-none-linux-gnueabihf for example). Can you please confirm that this passes successfully? This is ok from an arm perspective if testing is ok. Thanks, Kyrill
I had to change one testcase because it assumes that a divide by constant will always be emitted as a multiply. That actually depends on the relative costs of multiply, shift, and divide instructions. I ended up with a divide instruction for my target, as it has reasonably fast divide instructions. I fixed it by adding a -mtune=cortex-a53 option for aarch64 to ensure that we always get the multiply insn. Jim
