Re: [Mesa-dev] [PATCH] ac: fix possible truncation of intrinsic name

2018-06-07 Thread Bas Nieuwenhuizen
It doesn't fix a truncation though, as even though type is 8 chars we never write the full size (I think our max is 5 chars), but GCC is apparently not smart enough to see that. Reviewed-by: Bas Nieuwenhuizen On Thu, Jun 7, 2018 at 2:30 AM, Timothy Arceri wrote: > Fixes the gcc warning: > snpr

[Mesa-dev] [PATCH] ac: fix possible truncation of intrinsic name

2018-06-06 Thread Timothy Arceri
Fixes the gcc warning: snprintf’ output between 26 and 33 bytes into a destination of size 32 Fixes: d5f7ebda3ec0 ("ac: add LLVM build functions for subgroup instrinsics") --- src/amd/common/ac_llvm_build.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/amd/common/ac_llvm