On 20/05/2015 18:25, Richard Henderson wrote:

> 
>> +    int i;                                                              \
>> +    for (i = 0; i < DF_ELEMENTS(DF); i++) {                             \
>> +        wx.TYPE[i] = LD_INSN(env, addr + (i << DF), ##__VA_ARGS__);     \
> 
> Why the varargs?  They're the same all the time.

No, they aren't. we still need to support linux user mode which don't have
any extra argument.

MSA_LD_DF(DF_BYTE,   b, helper_ret_ldub_mmu, \
          make_memop_idx(MO_UNALN, cpu_mmu_index(env)), GETRA())

VS.

MSA_LD_DF(DF_BYTE,   b, cpu_ldub_data)


Regards,
Yongbok


Reply via email to