Does that do the right thing? It seems like this would introduce padding between "type" and "sample_mask" that previously wasn't there.
I only aligns `type`, so if the second member has 4-byte alignment there is no padding.
Right, my bad. Note that the latest version of the MR still has pipeline_state_desc_from_d3d12_graphics_desc() and pipeline_state_desc_from_d3d12_compute_desc() returning HRESULTs in the commits that introduce them though. Patch 4/6 gets rid of those, but ideally we wouldn't introduce them in the first place.