Skip to content
Permalink
Browse files

Merge pull request #8144 from lioncash/tie

Common/x64Emitter: Resolve TODO in OpArg's operator==
  • Loading branch information...
leoetlino committed May 31, 2019
2 parents 43eef41 + 66596c5 commit 5ffbec69561022a3244735ba8e98cb9ff81bef54
Showing with 2 additions and 4 deletions.
  1. +2 −4 Source/Core/Common/x64Emitter.h
@@ -120,10 +120,8 @@ struct OpArg
}
constexpr bool operator==(const OpArg& b) const
{
// TODO: Use std::tie here once Dolphin requires C++17. (We can't do it immediately,
// (because we still support some older versions of GCC where std::tie is not constexpr.)
return operandReg == b.operandReg && scale == b.scale && offsetOrBaseReg == b.offsetOrBaseReg &&
indexReg == b.indexReg && offset == b.offset;
return std::tie(scale, offsetOrBaseReg, indexReg, offset, operandReg) ==
std::tie(b.scale, b.offsetOrBaseReg, b.indexReg, b.offset, b.operandReg);
}
constexpr bool operator!=(const OpArg& b) const { return !operator==(b); }
u64 Imm64() const

0 comments on commit 5ffbec6

Please sign in to comment.
You can’t perform that action at this time.