On 30 August 2017 at 11:07, Lionel Landwerlin
<lionel.g.landwer...@intel.com> 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.
>
> 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
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
