diff options
| author | Connor Abbott <[email protected]> | 2019-08-29 17:15:46 +0200 |
|---|---|---|
| committer | Connor Abbott <[email protected]> | 2019-09-05 12:21:35 +0200 |
| commit | 5dadbabb47c317f58a3b939d08e8e50745d92990 (patch) | |
| tree | cd2c4e4a9d70605a8518a8adaa1dead773a1bf86 /src/gallium | |
| parent | 5cc7cc5f171079919dbe26f8e7a4d4eb53d119bb (diff) | |
radv/radeonsi: Don't count read-only data when reporting code size
We usually use these counts as a simple way to figure out if a change
reduces the number of instructions or shrinks an instruction. However,
since .rodata sections aren't executed, we shouldn't be counting their
size for this analysis. Make the linker return the total executable
size, and use it to report the more useful size in both drivers.
Reviewed-by: Marek Olšák <[email protected]>
Diffstat (limited to 'src/gallium')
| -rw-r--r-- | src/gallium/drivers/radeonsi/si_shader.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gallium/drivers/radeonsi/si_shader.c b/src/gallium/drivers/radeonsi/si_shader.c index cd75c084043..7636d44bee4 100644 --- a/src/gallium/drivers/radeonsi/si_shader.c +++ b/src/gallium/drivers/radeonsi/si_shader.c @@ -5286,7 +5286,7 @@ static unsigned si_get_shader_binary_size(struct si_screen *screen, struct si_sh { struct ac_rtld_binary rtld; si_shader_binary_open(screen, shader, &rtld); - return rtld.rx_size; + return rtld.exec_size; } static bool si_get_external_symbol(void *data, const char *name, uint64_t *value) |