On 06/21/2011 02:25 PM, Ramana Radhakrishnan wrote: > On 21 June 2011 22:04, Janis Johnson <jani...@codesourcery.com> wrote: >> This patch removes "-march=armv7-a" from four tests that pass with all >> -march values from armv4t on. >> >> OK for trunk? >> > > No, that sounds like a bug - if we are generating smlaltb and friends > for v4t then the compiler is broken. This is probably because smlaltb > is a part of the file name :( . > > smlaltb should be generated in ARM state for arch between v5e onwards > in Thumb state from v6t2 onwards (i.e. v6t2).
When I rename the files the tests fail for lots of -march values, so renaming the tests is definitely needed. The instructions checked for in these tests are provided in arm.md for TARGET_DSP_MULTIPLY; is there a general way to check for that, or should the test continue to provide a -march value that supports it and also skip -march values from multilib flags that are known to not support those instructions? Janis
