radeonsi: implement volatile memory access
authorNicolai Hähnle <nicolai.haehnle@amd.com>
Mon, 14 Mar 2016 15:22:21 +0000 (10:22 -0500)
committerNicolai Hähnle <nicolai.haehnle@amd.com>
Mon, 21 Mar 2016 20:34:25 +0000 (15:34 -0500)
Prevent loads from being re-ordered or coalesced.

Atomics don't need special handling by definition, and stores don't need
special handling because LLVM is unable to detect dead image or buffer
stores.

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
src/gallium/drivers/radeonsi/si_shader.c

index db0cc5bbf620e3873dd27e554d4ff81b418fc18e..0d26957101e519bd2f2522eb7ee1804d6eccf59f 100644 (file)
@@ -2915,6 +2915,7 @@ static void load_emit(
                struct lp_build_tgsi_context *bld_base,
                struct lp_build_emit_data *emit_data)
 {
+       struct si_shader_context *ctx = si_shader_context(bld_base);
        struct gallivm_state *gallivm = bld_base->base.gallivm;
        LLVMBuilderRef builder = gallivm->builder;
        const struct tgsi_full_instruction * inst = emit_data->inst;
@@ -2922,6 +2923,9 @@ static void load_emit(
        char intrinsic_name[32];
        char coords_type[8];
 
+       if (inst->Memory.Qualifier & TGSI_MEMORY_VOLATILE)
+               emit_optimization_barrier(ctx);
+
        if (target == TGSI_TEXTURE_BUFFER) {
                emit_data->output[emit_data->chan] = lp_build_intrinsic(
                        builder, "llvm.amdgcn.buffer.load.format.v4f32", emit_data->dst_type,