Quoted 'verify', removed spurious newlines.

Updated thanks

Update summary as I realized the doc was out of date but we had already fixed the bug about element type. Instead generalized broadcastable to variadic operands, rename trait and addressed comments.

Looks good thanks

I think you may have forgotten to upload the changed diff :) In general I think this looks good, I'd like to remove the duplicate functions but beyond that this should be good.

Looks good thanks

Yes the missing docs are considered static and supposed to be in mlir-www and the links to them work on mlir.llvm.org via relative, but could be made explicit to the website here..

Nice, seems to match what clang does here too so that's good to keep it in sync

Looks good on first walk through

Apparently I forgot to hit submit (seeing unsubmitted comments below).

Updated docs & switched to function_ref.

This is mostly a mechanical move or is there anything new to look at, correct?

LG, thanks

LG for Lanai changes too

Thanks!

I don't know this part as well as Mehdi does, but it looks good.

This seems to break on GCC 5.5:

Could we add a test that this would have hit? (I'm thinking of one just assigning the sliced/drop_front one).

Thanks

Thanks!

Looks good in general, thanks

Thanks

Looks good as part of optimizing the size, thanks