Code review comment for lp:~eres/gcc-linaro/sms_fix_mem_dep-4.6

Revision history for this message
Revital Eres (eres) wrote :

> The default bootstrap and test doesn't check sms. My understanding was that
> the original patches on FSF trunk made sure that trunk bootstrapped with SMS.

Right, the patches sent upstream are tested on ARM machine using bootstrap with SMS flags enabled.

> Should we regularly bootstrap with SMS turned on ?

If we have spare resources then yes. Please note that I usually do bootstrap on 4 different flavors of ARM, configuring with and without the following two flags --with-arch=armv7-a --with-mode=thumb. Also, I'm currently looking into another bootstrap failure on ARM machine exposed in recent trunk so once that will be resolved we can consider this issue again.

« Back to merge proposal