Comment 16 for bug 383240

Revision history for this message
Dave Martin (dave-martin-arm) wrote :

Will do— I think I now have a good patch set, but I want to check it works. I'm currently doing a fresh build.

One other question Mans:

Because the baseline architecture used by the toolchain is -march=armv5t, many of the NEON sources fail to build even with -mfpu=neon, because of the presence of PLD instructions (not supported in v5).

My current fix for this is to add -march=armv7-a in extra-cflags for the whole NEON-enabled build pass: my rationale for this is that the presence of NEON implies v7, which means that v7 features may as well be enabled for the whole pass. (This seems a general problem with -mfpu= and -march/cpu= which allow non-existent architectures, such as v5+NEON or NEON-without-v7 to be selected when used together.)

Is this the right thing to do, or do you have a different suggestion?