diff options
author | Jordan Justen <[email protected]> | 2018-07-25 14:31:05 -0700 |
---|---|---|
committer | Jordan Justen <[email protected]> | 2018-08-01 23:49:16 -0700 |
commit | 8fcdb71d8c9e6a0d9d9b70550bb764b654377714 (patch) | |
tree | 91ce70041ce4f5cf91697c7535f8b586e6fb6f84 /src/intel/compiler | |
parent | 3887700dfd7597fba654a4a713c274213a4a8755 (diff) |
intel/compiler: Add brw_get_compiler_config_value for disk cache
During code review, Jason pointed out that:
2b3064c0731 "i965, anv: Use INTEL_DEBUG for disk_cache driver flags"
Didn't account for INTEL_SCALER_* environment variables.
To fix this, let the compiler return the disk_cache driver flags.
Another possible fix would be to pull the INTEL_SCALER_* into
INTEL_DEBUG bits, but as we are currently using 41 of 64 bits, I
didn't think it was a good use of 4 more of these bits. (5 since
INTEL_PRECISE_TRIG needs to be accounted for as well.)
Cc: Jason Ekstrand <[email protected]>
Signed-off-by: Jordan Justen <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/intel/compiler')
-rw-r--r-- | src/intel/compiler/brw_compiler.c | 27 | ||||
-rw-r--r-- | src/intel/compiler/brw_compiler.h | 12 |
2 files changed, 39 insertions, 0 deletions
diff --git a/src/intel/compiler/brw_compiler.c b/src/intel/compiler/brw_compiler.c index 6480dbefbf6..6df9621fe42 100644 --- a/src/intel/compiler/brw_compiler.c +++ b/src/intel/compiler/brw_compiler.c @@ -181,6 +181,33 @@ brw_compiler_create(void *mem_ctx, const struct gen_device_info *devinfo) return compiler; } +static void +insert_u64_bit(uint64_t *val, bool add) +{ + *val = (*val << 1) | !!add; +} + +uint64_t +brw_get_compiler_config_value(const struct brw_compiler *compiler) +{ + uint64_t config = 0; + insert_u64_bit(&config, compiler->precise_trig); + if (compiler->devinfo->gen >= 8 && compiler->devinfo->gen < 10) { + insert_u64_bit(&config, compiler->scalar_stage[MESA_SHADER_VERTEX]); + insert_u64_bit(&config, compiler->scalar_stage[MESA_SHADER_TESS_CTRL]); + insert_u64_bit(&config, compiler->scalar_stage[MESA_SHADER_TESS_EVAL]); + insert_u64_bit(&config, compiler->scalar_stage[MESA_SHADER_GEOMETRY]); + } + uint64_t debug_bits = INTEL_DEBUG; + uint64_t mask = DEBUG_DISK_CACHE_MASK; + while (mask != 0) { + const uint64_t bit = 1ULL << (ffsll(mask) - 1); + insert_u64_bit(&config, (debug_bits & bit) != 0); + mask &= ~bit; + } + return config; +} + unsigned brw_prog_data_size(gl_shader_stage stage) { diff --git a/src/intel/compiler/brw_compiler.h b/src/intel/compiler/brw_compiler.h index 4797c9cf06d..c510d34ce2e 100644 --- a/src/intel/compiler/brw_compiler.h +++ b/src/intel/compiler/brw_compiler.h @@ -1214,6 +1214,18 @@ DEFINE_PROG_DATA_DOWNCAST(sf) struct brw_compiler * brw_compiler_create(void *mem_ctx, const struct gen_device_info *devinfo); +/** + * Returns a compiler configuration for use with disk shader cache + * + * This value only needs to change for settings that can cause different + * program generation between two runs on the same hardware. + * + * For example, it doesn't need to be different for gen 8 and gen 9 hardware, + * but it does need to be different if INTEL_DEBUG=nocompact is or isn't used. + */ +uint64_t +brw_get_compiler_config_value(const struct brw_compiler *compiler); + unsigned brw_prog_data_size(gl_shader_stage stage); |