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