[llvm-commits] [llvm] r45126 - /llvm/trunk/include/llvm/Target/TargetInstrInfo.h

[Bill Wendling]

On Dec 17, 2007 3:15 PM, Chris Lattner <clattner at apple.com> wrote:
>
> > +  /// hasSideEffects - Returns true if the instruction has side
> > effects that are
> > +  /// not captured by any operands of the instruction or other flags.
> > +  bool hasSideEffects(MachineInstr *MI) const {
>
> How about "hasUnmodelledSideEffects() ?
>
> I would expect a method named 'hasSideEffects' to return true if
> flags/operands indicate a side effect.
>
Good point. I'll make the change. :-)

-bw