[PATCH] Fix undefined behaviour in lrint,llrint.

Dave Korn dave.korn.cygwin@gmail.com
Sat Jul 31 00:36:00 GMT 2010

On 27/07/2010 00:59, Howland Craig D (Craig) wrote:
> As I had pointed out in the email thread concerning (possible) undefined
> shift behavior in lround and llround
> (http://sourceware.org/ml/newlib/2010/msg00437.html), lrint and llrint
> both have the same potential issue.  Attached is a patch--based on
> Dave's edits of lround--that fixes the problem.  The patch also corrects
> a minor typo in the comments for the previous lround and llround edits.

  Thanks for doing these functions (and saving me from having to), and for
catching my typo.  It all looked right to me but I didn't actually test it.


More information about the Newlib mailing list