On Thu, Jan 4, 2018 at 4:00 AM, Mark Rutland <mark.rutland@xxxxxxx> wrote: > On Wed, Jan 03, 2018 at 10:38:24PM +0000, Mark Rutland wrote: >> +#define nospec_array_load(arr, idx, sz) \ >> +({ \ >> + typeof(*(arr)) *__arr = arr; \ >> + typeof(idx) __idx = idx; \ >> + typeof(sz) __sz = __sz; \ > > Whoops. The second __sz should be sz here. Those should all have parenthesis on the args too, right?