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

Dale Johannesen dalej at apple.com
Mon Jan 19 15:19:26 PST 2009


On Jan 19, 2009, at 3:09 PMPST, Neil Booth wrote:

> bugzilla-daemon at cs.uiuc.edu wrote:-
>
>> http://llvm.org/bugs/show_bug.cgi?id=3316
>>
>> --- 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.

Ah, that's what TRT means.
Could you be more specific about what doesn't work (when you have time)?




More information about the llvm-bugs mailing list