On Thu, Jan 04, 2018 at 08:21:56PM -0800, Dan Williams wrote:
> 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?
Probably, yes.
I've added those to the version in my core/nospec branch.
Thanks,
Mark.
