On Dec 17, 2007 3:15 PM, Chris Lattner <[EMAIL PROTECTED]> 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
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to