This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] aarch64 sim bug fix for fcmp and infinity


Hi Jim,

> Debugging another gcc testsuite failure, I see that fcmp can fail when
> both operands are infinities.  The code tries to compute the compare
> result by subtracting operands, but this can yield NaN when both
> operands are inifinities.  This is fixed by explicitly checking for
> and handling this case.
> 
> While writing the tescase, I noticed that qNan and sNan are not
> handled correctly, but this appears to be broken in more than just the
> compare instructions, so I just added comments to document the problem
> and will leave it to a later patch to fix.
> 
> The new testcase fails without the patch, and works with the patch.
> Also, the gcc C testsuite unexpected failures drop  from 2627 to 2568.
 
Approved - please apply - and thanks for the continued testing.

Cheers
  Nick


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]