llvm icon indicating copy to clipboard operation
llvm copied to clipboard

Add SPV_INTEL_float_controls2 extension preview

Open mmerecki opened this issue 5 years ago • 4 comments

Signed-off-by: Mariusz Merecki [email protected]

mmerecki avatar Apr 29 '20 13:04 mmerecki

@bashbaug @AlexeySotkin @kvladimi @aus-intel could you please take a look at this MR?

mmerecki avatar Apr 30 '20 09:04 mmerecki

@AlexeySachkov, @bader - when we can expect this extension to be reviewed/merged? Some of ESIMD features depend on this.

kbobrovs avatar Apr 04 '22 16:04 kbobrovs

Have you given any thought how similar float controls could apply to bfloat16 or tf32 types, which would have the same "target width" as float and half types respectively?

yes, I thought we could replace "target width" with an enum whose values would be float = 32, half = 16, double = 64 and something new for bfloat16 or tf32. Such change would be backwards compatible.

mmerecki avatar Sep 14 '22 11:09 mmerecki

yes, I thought we could replace "target width" with an enum whose values would be float = 32, half = 16, double = 64 and something new for bfloat16 or tf32. Such change would be backwards compatible.

Nice, this seems reasonable.

Since the "target width" is an unsigned 32-bit integer maybe it makes sense to choose absurdly large values (e.g. with the high bit set) to represent alternate floating-point encodings like bfloat16 or tf32?

bashbaug avatar Sep 15 '22 23:09 bashbaug

This pull request is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be automatically closed in 30 days.

github-actions[bot] avatar Mar 16 '23 02:03 github-actions[bot]

This pull request was closed because it has been stalled for 30 days with no activity.

github-actions[bot] avatar Apr 16 '23 02:04 github-actions[bot]

@mmerecki @bashbaug hi, do you know, what is the status here?

MrSidims avatar Nov 15 '23 13:11 MrSidims