[LLVMbugs] [Bug 3316] Constant folding for 'frem' is flaky

Neil Booth neil at daikokuya.co.uk
Mon Jan 19 15:09:45 PST 2009


bugzilla-daemon at cs.uiuc.edu wrote:-

> http://llvm.org/bugs/show_bug.cgi?id=3316
> 
> 
> Dale Johannesen <dalej at apple.com> changed:
> 
>            What    |Removed                     |Added
> ----------------------------------------------------------------------------
>              Status|NEW                         |RESOLVED
>          Resolution|                            |FIXED
> 
> 
> 
> 
> --- Comment #6 from Dale Johannesen <dalej at apple.com>  2009-01-19 15:20:58 ---
> I see, currently the FP-to-integer conversions are always called with
> TowardsZero, except this one place.  So the bug calling it with
> NearestTiesToEven would never show up anywhere else.  This fixes both bugs.
> Neil, please review.
> http://llvm.org/viewvc/llvm-project?view=rev&revision=62528

I'll take a look later; no time now.  But I left a comment in / above
the frem implementation pointing out that it isn't close to being
correct.  It will need a complete rewrite and will have to look like
the other operations before it can be called correct.

Neil.



More information about the llvm-bugs mailing list