Hi Segher,

>>  proc check_effective_target_vect_len_load_store { } {
>> -    return 0
>> +    return [expr { [check_effective_target_has_arch_pwr9] }]
>>  }
> 
> Why not just
> 
>   return check_effective_target_has_arch_pwr9;
> 
> ?  (Or lose at least two pairs of brackets if not all three :-) )
> 

Thanks for the suggestion to make it concise!

I tried
  
    return check_effective_target_has_arch_pwr9

it failed due to that:
   
    ERROR:... error executing dg-final: expected boolean value but got 
"check_effective_target_has_arch_pwr9"

I re-tried
    
    return [check_effective_target_has_arch_pwr9]

it worked, nice!

Committed in r11-2960 with this concise writing then.

BR,
Kewen

Reply via email to