gboisse / gfx

A minimalist and easy to use graphics API.
MIT License
498 stars 35 forks source link

Support additional include paths for kernels #119

Closed VladimirMakeew closed 5 months ago

VladimirMakeew commented 6 months ago

Allow users to pass additional shader include search paths when creating mesh, compute, graphics or raytracing kernels. Parameters are optional, same as with defines.

maoliver-amd commented 6 months ago

Would it make sense to add the include paths in at the program level instead of kernel? Programs already have the shader path so include paths may make more sense there. This also reduces the number of places that include paths need to be added as its just in one location. This is more of a design decision so @gboisse what do you think?

VladimirMakeew commented 5 months ago

Discussed it with @gboisse and we decided to pass additional shader include paths to programs instead of kernels.