Clang throws warnings here (-Wgnu-variable-sized-type-not-at-end) because of a variable-sized array at the end of `drm_i915_query_topology_info` being aliased by the `data` field in `topology`, but that's the point here, so let's explain it so that the next me doesn't blindly listen to the compiler :)
Suggested-by: Brian Paul <bri...@vmware.com> Signed-off-by: Eric Engestrom <eric.engest...@intel.com> --- src/intel/dev/gen_device_info.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/intel/dev/gen_device_info.c b/src/intel/dev/gen_device_info.c index 5dbd06075722f8cc644e..1be76f7bf6fa9da346a0 100644 --- a/src/intel/dev/gen_device_info.c +++ b/src/intel/dev/gen_device_info.c @@ -990,6 +990,11 @@ gen_device_info_update_from_masks(struct gen_device_info *devinfo, uint32_t subslice_mask, uint32_t n_eus) { + /* + * Note: the `data` field here is the necessary allocation for the + * zero-length `data` field in drm_i915_query_topology_info. + * Do not reorder! + */ struct { struct drm_i915_query_topology_info base; uint8_t data[100]; -- Cheers, Eric _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev