[llvm-commits] [llvm] r75567 - /llvm/trunk/lib/VMCore/Instruction.cpp

[Dan Gohman]

On Jul 16, 2009, at 1:46 PM, Eli Friedman wrote:


> On Thu, Jul 16, 2009 at 1:28 PM, Dan Gohman<gohman at apple.com> wrote:
>
>> The isSafeToSpeculativelyExecute in this patch misses
>>
>> code for FRem and FDiv.
>>
>
> Is there some reason why it isn't safe to execute them speculatively?

Oops, no. They're in the current isTrapping() list, but it looks
like that's an artifact of the way that Div was split into
SDiv/UDiv/FDiv. There are processor modes where FDiv etc. can
trap, but LLVM doesn't know about those right now.

Dan