diff options
| author | Kenneth Graunke <[email protected]> | 2018-06-07 15:44:43 -0700 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2018-06-11 14:55:32 -0700 |
| commit | 0d5329d626e3f96a7788880052ae2a5ecfc8cdbe (patch) | |
| tree | 62d3344f2cab0c618c466e2e76a360877c206808 /src | |
| parent | 06e8771decaad62089e1c7e75905683ae12bc12f (diff) | |
anv: Disable __gen_validate_value if NDEBUG is set.
We were enabling undefined memory checking for genxml values based on
Valgrind being installed at build time, even for release builds. This
generates piles and piles of assembly whenever you touch genxml.
With gcc 7.3.1 and -O3 and -march=native on a Kabylake with Valgrind
installed at build time:
text data bss dec hex filename
5978385 262884 13488 6254757 5f70a5 libvulkan_intel.so
3799377 262884 13488 4075749 3e30e5 libvulkan_intel.so
That's a 36% reduction in text size.
Fixes: 047ed02723071d7eccbed3210b5be6ae73603a53 (vk/emit: Use valgrind to validate every packed field)
Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src')
| -rw-r--r-- | src/intel/vulkan/anv_private.h | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h index b47420f89ec..b035bf1c943 100644 --- a/src/intel/vulkan/anv_private.h +++ b/src/intel/vulkan/anv_private.h @@ -36,7 +36,9 @@ #include <valgrind.h> #include <memcheck.h> #define VG(x) x +#ifndef NDEBUG #define __gen_validate_value(x) VALGRIND_CHECK_MEM_IS_DEFINED(&(x), sizeof(x)) +#endif #else #define VG(x) #endif |