On Dec 17, 2007, at 1:53 PM, Bill Wendling wrote: > > ====================================================================== > ======== > --- llvm/trunk/include/llvm/Target/TargetInstrInfo.h (original) > +++ llvm/trunk/include/llvm/Target/TargetInstrInfo.h Mon Dec 17 > 15:53:30 2007 > @@ -314,6 +314,15 @@ > isReallyTriviallyReMaterializable(MI); > } > > + /// 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. -Chris _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits