[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