This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH] Fix undefined behaviour in lrint,llrint.
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.
cheers,
DaveK