> -----Original Message----- > From: Taylor Simpson > Sent: Thursday, April 27, 2023 6:05 PM > To: Richard Henderson <[email protected]>; Paolo Bonzini > <[email protected]>; [email protected] > Subject: RE: [PATCH] target/hexagon: fix = vs. == mishap > > > > > -----Original Message----- > > From: Richard Henderson <[email protected]> > > Sent: Thursday, April 27, 2023 8:33 AM > > To: Paolo Bonzini <[email protected]>; [email protected] > > Cc: Taylor Simpson <[email protected]> > > Subject: Re: [PATCH] target/hexagon: fix = vs. == mishap > > > > WARNING: This email originated from outside of Qualcomm. Please be > wary > > of any links or attachments, and do not enable macros. > > > > On 4/27/23 13:56, Paolo Bonzini wrote: > > > Coverity reports a parameter that is "set but never used". This is > > > caused by an assignment operator being used instead of equality. > > > > > > Cc: Taylor Simpson<[email protected]> > > > Signed-off-by: Paolo Bonzini<[email protected]> > > > --- > > > target/hexagon/idef-parser/parser-helpers.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Reviewed-by: Richard Henderson <[email protected]> > > Queued with next Hexagon update
This patch is causing some yyassert's when idef-parser runs. They are coming from sign and zero extends. WARNING (A2_vavgw): 'Extending to a size smaller than the current size makes no sense' I will investigate. Taylor
