On 11/07/2011 10:54 PM, Tom Tromey wrote:
> Yao> +proc support_process_record {} {
> Yao> +
> Yao> +    if { [istarget "x86_64-*-linux*"] || [istarget "i\[34567\]86-*-linux*"] } {
> 
> It seems to me that both of the new procs should include the old check
> using target_info as well, just in case.  This will avoid breaking
> anybody's existing setup.

Right.  In new patch, target_info is honored in proc, so users can
disable reverse tests in their own board file (to Joel's comment).  Two
procs are renamed from support_FOO to suppors_FOO (as Pedro pointed out).

> 
> Ok with that change.

This is what I committed.

-- 
Yao (齐尧)