>    int structstkspace;
>>    int argoffset;
>>    int structoffset;
>> -  struct value *arg;
>>    struct type *type;
>>    int len;
>>    char old_sp_buf[4];
>>    CORE_ADDR saved_sp;
>> +  struct gdbarch_tdep *tdep = gdbarch_tdep (current_gdbarch);
>> +  const struct bfd_arch_info *arch_info = gdbarch_bfd_arch_info (current_gdbarch);
> 
> 
> Now that you've introduced ppc_floating_point_unit_p(), I don't
> think that arch_info is needed any longer.

Good point (need to get -Wunused-local building).  Removed.

> Otherwise, okay.

Checked in the attached.

Andrew