Skip to content

Commit

Permalink
Remove unnecessary "inline" of inline defined member functions
Browse files Browse the repository at this point in the history
Member functions defined within a class definition are implicitly
'inline' for linkage purposes. Compilers might slightly favor inlining
functions explicitly marked 'inline', but LLVM doesn't make a stylistic
habit of doing this generally.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@205679 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
dwblaikie committed Apr 5, 2014
1 parent 2a65dd3 commit 553e400
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions include/llvm/CodeGen/MachineInstr.h
Original file line number Diff line number Diff line change
Expand Up @@ -287,10 +287,10 @@ class MachineInstr : public ilist_node<MachineInstr> {
const_mop_iterator operands_begin() const { return Operands; }
const_mop_iterator operands_end() const { return Operands + NumOperands; }

inline iterator_range<mop_iterator> operands() {
iterator_range<mop_iterator> operands() {
return iterator_range<mop_iterator>(operands_begin(), operands_end());
}
inline iterator_range<const_mop_iterator> operands() const {
iterator_range<const_mop_iterator> operands() const {
return iterator_range<const_mop_iterator>(operands_begin(), operands_end());
}

Expand All @@ -299,10 +299,10 @@ class MachineInstr : public ilist_node<MachineInstr> {
mmo_iterator memoperands_end() const { return MemRefs + NumMemRefs; }
bool memoperands_empty() const { return NumMemRefs == 0; }

inline iterator_range<mmo_iterator> memoperands() {
iterator_range<mmo_iterator> memoperands() {
return iterator_range<mmo_iterator>(memoperands_begin(), memoperands_end());
}
inline iterator_range<mmo_iterator> memoperands() const {
iterator_range<mmo_iterator> memoperands() const {
return iterator_range<mmo_iterator>(memoperands_begin(), memoperands_end());
}

Expand Down

0 comments on commit 553e400

Please sign in to comment.