ac/nir: set GLC=1 for load/store of coherent/volatile images
This disables persistence accross wavefronts. F1 2017 and Wolfenstein 2 appear to use some coherent images but this patch doesn't seem to change anything. Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
This commit is contained in:
parent
3c40be126f
commit
a01e9996b5
|
@ -3657,13 +3657,13 @@ static LLVMValueRef visit_image_load(struct ac_nir_context *ctx,
|
|||
res = ac_to_integer(&ctx->ac, res);
|
||||
} else {
|
||||
LLVMValueRef da = glsl_is_array_image(type) ? ctx->ac.i1true : ctx->ac.i1false;
|
||||
LLVMValueRef glc = ctx->ac.i1false;
|
||||
LLVMValueRef slc = ctx->ac.i1false;
|
||||
|
||||
params[0] = get_image_coords(ctx, instr);
|
||||
params[1] = get_sampler_desc(ctx, instr->variables[0], AC_DESC_IMAGE, NULL, true, false);
|
||||
params[2] = LLVMConstInt(ctx->ac.i32, 15, false); /* dmask */
|
||||
params[3] = glc;
|
||||
params[3] = (var->data.image._volatile || var->data.image.coherent) ?
|
||||
ctx->ac.i1true : ctx->ac.i1false;
|
||||
params[4] = slc;
|
||||
params[5] = ctx->ac.i1false;
|
||||
params[6] = da;
|
||||
|
@ -3711,7 +3711,8 @@ static void visit_image_store(struct ac_nir_context *ctx,
|
|||
params[1] = get_image_coords(ctx, instr); /* coords */
|
||||
params[2] = get_sampler_desc(ctx, instr->variables[0], AC_DESC_IMAGE, NULL, true, true);
|
||||
params[3] = LLVMConstInt(ctx->ac.i32, 15, false); /* dmask */
|
||||
params[4] = glc;
|
||||
params[4] = (force_glc || var->data.image._volatile || var->data.image.coherent) ?
|
||||
ctx->ac.i1true : ctx->ac.i1false;
|
||||
params[5] = slc;
|
||||
params[6] = ctx->ac.i1false;
|
||||
params[7] = da;
|
||||
|
|
Loading…
Reference in New Issue