radeonsi: add a debug flag that disables optimized shader variants
authorMarek Olšák <marek.olsak@amd.com>
Mon, 21 Nov 2016 19:39:27 +0000 (20:39 +0100)
committerMarek Olšák <marek.olsak@amd.com>
Wed, 23 Nov 2016 17:49:10 +0000 (18:49 +0100)
Reviewed-by: Edward O'Callaghan <funfunctor@folklore1984.net>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
src/gallium/drivers/radeon/r600_pipe_common.c
src/gallium/drivers/radeon/r600_pipe_common.h
src/gallium/drivers/radeonsi/si_state_shaders.c

index 1a7a712beb2121de5fb50563ddb3e5ec58092e0b..15e8a12647ef5b650829c5aa26a795ae75a446a2 100644 (file)
@@ -671,6 +671,7 @@ static const struct debug_named_value common_debug_options[] = {
        { "noasm", DBG_NO_ASM, "Don't print disassembled shaders"},
        { "preoptir", DBG_PREOPT_IR, "Print the LLVM IR before initial optimizations" },
        { "checkir", DBG_CHECK_IR, "Enable additional sanity checks on shader IR" },
+       { "nooptvariant", DBG_NO_OPT_VARIANT, "Disable compiling optimized shader variants." },
 
        { "testdma", DBG_TEST_DMA, "Invoke SDMA tests and exit." },
 
index 941fc377a514121c092fbb7ccd12a37a81545390..25d40da931e74d44d267a11c37b5c3617b1ee5a6 100644 (file)
@@ -79,6 +79,7 @@
 #define DBG_NO_ASM             (1 << 14)
 #define DBG_PREOPT_IR          (1 << 15)
 #define DBG_CHECK_IR           (1 << 16)
+#define DBG_NO_OPT_VARIANT     (1 << 17)
 /* gaps */
 #define DBG_TEST_DMA           (1 << 20)
 /* Bits 21-31 are reserved for the r600g driver. */
index fd0be30065053a62e042a0e8214bf99fa4effa31..0afc3b4fc9d0fb585b8120632b1f28be81a61e0c 100644 (file)
@@ -1121,6 +1121,11 @@ static int si_shader_select_with_key(struct si_screen *sscreen,
        struct si_shader_selector *sel = state->cso;
        struct si_shader *current = state->current;
        struct si_shader *iter, *shader = NULL;
+
+       if (unlikely(sscreen->b.chip_class & DBG_NO_OPT_VARIANT)) {
+               memset(&key->opt, 0, sizeof(key->opt));
+       }
+
 again:
        /* Check if we don't need to change anything.
         * This path is also used for most shaders that don't need multiple