Merge pull request #6645 from JosJuice/tie-constexpr

x64Emitter: Don't assume that std::tie is constexpr
This commit is contained in:
Mat M 2018-04-15 03:11:31 -04:00 committed by GitHub
commit eecdb51709
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions

View File

@ -120,8 +120,10 @@ struct OpArg
} }
constexpr bool operator==(const OpArg& b) const constexpr bool operator==(const OpArg& b) const
{ {
return std::tie(scale, offsetOrBaseReg, indexReg, offset, operandReg) == // TODO: Use std::tie here once Dolphin requires C++17. (We can't do it immediately,
std::tie(b.scale, b.offsetOrBaseReg, b.indexReg, b.offset, b.operandReg); // (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;
} }
constexpr bool operator!=(const OpArg& b) const { return !operator==(b); } constexpr bool operator!=(const OpArg& b) const { return !operator==(b); }
u64 Imm64() const u64 Imm64() const