diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp
index 67a5a8c0509..c6683ac6916 100644
--- a/src/compiler/glsl/ast_to_hir.cpp
+++ b/src/compiler/glsl/ast_to_hir.cpp
@@ -7812,10 +7812,12 @@ ast_interface_block::hir(exec_list *instructions,
    }
 
    unsigned qual_xfb_buffer;
-   if (!process_qualifier_constant(state, &loc, "xfb_buffer",
-                                   layout.xfb_buffer, &qual_xfb_buffer) ||
-       !validate_xfb_buffer_qualifier(&loc, state, qual_xfb_buffer)) {
-      return NULL;
+   if (layout.flags.q.xfb_buffer) {
+      if (!process_qualifier_constant(state, &loc, "xfb_buffer",
+                                      layout.xfb_buffer, &qual_xfb_buffer) ||
+          !validate_xfb_buffer_qualifier(&loc, state, qual_xfb_buffer)) {
+         return NULL;
+      }
    }
 
    unsigned qual_xfb_offset;

This makes it go away, but I'm unsure if it's the right thing to do... ```patch diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp index 67a5a8c0509..c6683ac6916 100644 --- a/src/compiler/glsl/ast_to_hir.cpp +++ b/src/compiler/glsl/ast_to_hir.cpp @@ -7812,10 +7812,12 @@ ast_interface_block::hir(exec_list *instructions, } unsigned qual_xfb_buffer; - if (!process_qualifier_constant(state, &loc, "xfb_buffer", - layout.xfb_buffer, &qual_xfb_buffer) || - !validate_xfb_buffer_qualifier(&loc, state, qual_xfb_buffer)) { - return NULL; + if (layout.flags.q.xfb_buffer) { + if (!process_qualifier_constant(state, &loc, "xfb_buffer", + layout.xfb_buffer, &qual_xfb_buffer) || + !validate_xfb_buffer_qualifier(&loc, state, qual_xfb_buffer)) { + return NULL; + } } unsigned qual_xfb_offset; ```