Re: [dpdk-dev] [PATCH] common/cnxk: fix build failure

[Ferruh Yigit <ferruh.yigit@intel.com>]

On 10/27/2021 2:12 PM, Tejasree Kondoj wrote:
> Fixing build failure with EXTRA_CFLAGS='-O1'.
> 

For future, can you please provide a little more detail in the commit
log related to the build error,
like which compiler or compiler version cause the issue, what is the
build error and put part of the log in commit log that can help to
people searching for the error log.
Plus if the build error is genuine build error, or if you are just
silencing compiler false positive.

> Fixes: d85f9749f915 ("common/cnxk: add hash generation API")
> 
> Reported-by: Longfeng Liang <longfengx.liang@intel.com>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> ---
>   drivers/common/cnxk/roc_hash.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/common/cnxk/roc_hash.c b/drivers/common/cnxk/roc_hash.c
> index 092286e41e..4a34c7fbf8 100644
> --- a/drivers/common/cnxk/roc_hash.c
> +++ b/drivers/common/cnxk/roc_hash.c
> @@ -23,7 +23,7 @@ roc_hash_sha1_gen(uint8_t *msg, uint32_t *hash)
>   			       0x67452301, 0xEFCDAB89, 0x98BADCFE, 0x10325476,
>   			       0xC3D2E1F0};
>   	int i;
> -	uint32_t temp;		/* Temporary word value */
> +	uint32_t temp = 0;	/* Temporary word value */
>   	uint32_t W[80];		/* Word sequence */
>   	uint32_t A, B, C, D, E; /* Word buffers */
>   
>