diff options
| author | Samuel Pitoiset <[email protected]> | 2018-02-22 10:25:38 +0100 |
|---|---|---|
| committer | Samuel Pitoiset <[email protected]> | 2018-02-22 20:39:55 +0100 |
| commit | a01e9996b590a51b3a699b7f6dddda7fc866b362 (patch) | |
| tree | 56062b6901ea1178ee85244dfb84eb6149aae60b /src | |
| parent | 3c40be126fcd82ad29fa39ac3a4df6dd67da27e0 (diff) | |
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 <[email protected]>
Reviewed-by: Bas Nieuwenhuizen <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/amd/common/ac_nir_to_llvm.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c index 213306322dd..99a3471305b 100644 --- a/src/amd/common/ac_nir_to_llvm.c +++ b/src/amd/common/ac_nir_to_llvm.c @@ -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; |