spec source for cl_khr_kernel_clock

[bashbaug]

No description provided.

[bashbaug]

TODOS:

• Clarify that this is a provisional extension.
• Remove "ext" from feature test macros.
• Add undefined behavior sentence to SPIR-V environment spec.

[bashbaug]

I've fixed all of the TODOs listed above. I think this is ready to go.

[oddhack]

Aside from minor comments, LGTM from a markup perspective. I am a little confused though, because I thought this extension was in the recently ratified as non-provisional list that Neil sent out (and was thus surprised not to see an appendix for it when constructing #1113).

[oddhack]

(and was thus surprised not to see an appendix for it when constructing #1113).

To be clear, I am looking at the minutes from March 19th which have a "Ratification #333 - The following extensions are now ratified as final specifications" section including cl_khr_kernel_clock, so I assumed this had previously been published as provisional and wondered where the appendix was. I haven't looked into it any more deeply than that.

[bashbaug]

I am a little confused though, because I thought this extension was in the recently ratified as non-provisional list that Neil sent out

Yeah, I was initially confused by this too. The change to a provisional ratification was done at the last minute.

The ratification review package clearly says provisional though, so I believe this is correct. Regardless, I think it would be easier to change from provisional to final than final to provisional.

[bashbaug]

Agreed to merge in principle after editorial review.

[alycm]

LGTM

[bashbaug]

Thanks, merging. Please file a GitHub issue if any other issues are found, editorial or otherwise.

[bcalidas]

Extension text looks good.