[Mesa-dev] [PATCH 7/9] i965: ensure execution of fragment shader when fragment shader has atomic buffer access
Rogovin, Kevin
kevin.rogovin at intel.com
Wed May 6 03:35:54 PDT 2015
Hi,
> I think this check should be put in a utility function up in core Mesa somewhere. It's open-coded twice in this patch, and the check will change when GL_ARB_image_load_store lands.
Would a check of the form:
inline bool
_mesa_has_atomic_ops(struct gl_context *ctx)
{
return ctx-> Shader._CurrentFragmentProgram != NULL &&
ctx->Shader._CurrentFragmentProgram->NumAtomicBuffers > 0;
}
be good?
I am hesitant to make a check that just tests for side effects since for some hardware (like Gen8 for example) there is not just a simple single flag to say "must execute frag shader", instead there are several such flags that "force to run fragment shader", but the flags by themselves take on other meanings; there was some discussion on flag things in the first posting of this patch with Curro and Kenneth.
-Kevin
More information about the mesa-dev
mailing list