gpuweb / gpuweb

Where the GPU for the Web work happens!

Home Page:http://webgpu.io

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Resurrect vertex formats dropped for macOS < 10.13

kainino0x opened this issue · comments

In #231 we dropped a bunch of vertex formats that required macOS 10.13+.

In #1069 (comment), a year ago, we decided to require 10.13.

I knew we were missing some random stuff that we should have added back when we decided that, but that's a pretty major one. (It was not even 2 years a lot of years in between the two.)

We should add these back. List but with names updated:

MTLVertexFormatUChar --> uint8
MTLVertexFormatChar --> sint8
MTLVertexFormatUCharNormalized --> unorm8
MTLVertexFormatCharNormalized --> snorm8

MTLVertexFormatUShort --> uint16
MTLVertexFormatShort --> sint16
MTLVertexFormatUShortNormalized --> unorm16
MTLVertexFormatShortNormalized --> snorm16
MTLVertexFormatHalf --> float16

MTLVertexFormatUChar4Normalized_bgra --> unorm8x4-bgra / unorm8bgra (spelling?)

All of these are required in Vulkan 1.0 and D3D FL11_0.

(The 3-component formats we removed for D3D12 still can't be added back.)

In #508 (comment) we decided to require 10.13.

Sorry, wrong link. Correct one is #1069 (comment)
(It was more years later.)

GPU Web WG 2024-03-27
  • CW: seems easy except for one tiny name bikeshedding.
  • KN: dropped these a year ago - no reason any more to continue excluding them.
  • CW: one q, how do we name the unorm8bgra?
  • KN: regular one's unorm8x4.
  • KG: if other one's x4, should call this one x4.
  • CW: unorm8x4-bgra seems clear.
  • Agreed all around.