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

[Eli Friedman]

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?

> Also, instead of being conservative with
> void-typed instructions, it'd be better to just add the handful of
> instruction kinds to the switch. It looks like it'd just be Store,
> Free, and the terminators.

Okay, will do.

-Eli