On 30/08/17 11:38, Emil Velikov wrote:
On 30 August 2017 at 11:07, Lionel Landwerlin <[email protected]> wrote:@@ -249,6 +249,7 @@ static mesa_format intel_renderbuffer_format(struct gl_context * ctx, GLenum internalFormat) { struct brw_context *brw = brw_context(ctx); + const struct gen_device_info *devinfo = &brw->screen->devinfo;Since this is used only in the assert below, annotate it as MAYBE_UNUSED/UNUSED? Otherwise GCC will warn on release builds.
Sure, thanks!
switch (internalFormat) { default: @@ -270,7 +271,7 @@ intel_renderbuffer_format(struct gl_context * ctx, GLenum internalFormat) if (brw->has_separate_stencil) { return MESA_FORMAT_S_UINT8; } else { - assert(!brw->must_use_separate_stencil); + assert(!devinfo->must_use_separate_stencil); return MESA_FORMAT_Z24_UNORM_S8_UINT; } }-Emil
_______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
