It's fine for me to have this field, but once we decide to have it it would probably make sense to keep it for the whole life of the `struct hlsl_deref`, not just when storing the offset. I don't know if it can be useful before that, but the information it conveys make sense at any time, so by artificially restricting when it is available just makes our life harder.
I implemented a patch with this idea https://gitlab.winehq.org/fcasas/vkd3d/-/commit/4f2e06da2ce02ba6ed803cb793d1... , but I think it its controversial enough to better not include it in this MR.
Also, let's just call it `type` or `data_type`.
Done!