Shader Extension Proposal #202
Conversation
Co-authored-by: Ethan Lee <flibitijibibo@gmail.com> Co-authored-by: Caleb Cornett <caleb.cornett@outlook.com>
| @@ -166,7 +166,8 @@ typedef enum FNA3D_CubeMapFace | |||
| typedef enum FNA3D_BufferUsage | |||
| { | |||
| FNA3D_BUFFERUSAGE_NONE, | |||
| FNA3D_BUFFERUSAGE_WRITEONLY | |||
| FNA3D_BUFFERUSAGE_WRITEONLY, | |||
| FNA3D_BUFFERUSAGE_COMPUTE_EXT /* SDL shader extension only */ | |||
There was a problem hiding this comment.
Does this mean 'usable by compute AND write only'? Or is it a bit
There was a problem hiding this comment.
This value is actually completely ignored on every backend, so setting COMPUTE_EXT on the SDL backend is the only place this would actually matter.
include/FNA3D.h
Outdated
| */ | ||
| FNA3DAPI void FNA3D_BindComputeTexturesEXT( | ||
| FNA3D_Device *device, | ||
| SDL_GpuComputeTextureBinding *pBindings |
There was a problem hiding this comment.
Is there a blessed way to feed FNA3D textures in to this? Or would you use a SysTexture?
From looking at this it seems like in this proposal, fragment/vertex shaders ('graphics' shaders) can only consume FNA3D textures, and compute shaders can only consume raw SDL_Gpu textures.
There was a problem hiding this comment.
FNA3D texture handles are just sdl_gpu texture handles, but I agree that this is awkward... I'll just wrap the SDL struct and handle it in the implementation
There was a problem hiding this comment.
Actually I was wrong and FNA3D_Driver_SDL was wrapping the texture handles. So nice catch!
flibitijibibo
force-pushed
the
sdl_gpu
branch
7 times, most recently
from
a7054d7
to
a7138eb
Compare
flibitijibibo
force-pushed
the
sdl_gpu
branch
3 times, most recently
from
5cddd4b
to
2886b44
Compare
flibitijibibo
force-pushed
the
sdl_gpu
branch
3 times, most recently
from
ddc9f3d
to
937d457
Compare
flibitijibibo
force-pushed
the
sdl_gpu
branch
2 times, most recently
from
14b85f9
to
be2c255
Compare
flibitijibibo
force-pushed
the
sdl_gpu
branch
3 times, most recently
from
95a9a83
to
060febd
Compare
flibitijibibo
force-pushed
the
sdl_gpu
branch
3 times, most recently
from
6c45739
to
4fb962d
Compare
Tried to create something that would be as minimally invasive to the rest of the API as possible. When using the graphics shader extension, the rest of the API is still used normally, only the Effect calls are omitted.
Technically this doesn't break ABI because isRenderTarget on CreateTexture_ is now interpreted as a bitflag, but the value was always 1 or 0 before.