SwitchEmu/dynarmic
1
0
Fork 0
mirror of https://git.suyu.dev/suyu/dynarmic.git synced 2026-03-24 22:22:58 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #397 from VelocityRa/dec-shift-fix

Browse Source 
decoders: Cast to correctly-sized type before shifting
This commit is contained in:
Mat M
2018-09-22 19:17:36 -04:00
committed by MerryMage
parent 238f2f2cd0 c30b8dbe99
commit 9130ed45b9
1 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/frontend/decoder/decoder_detail.h
View File

@@ -36,16 +36,17 @@ private:
* A '1' in a bitstring indicates that a one must be present at that bit position.
*/
static auto GetMaskAndExpect(const char* const bitstring) {
const auto one = static_cast<opcode_type>(1);
opcode_type mask = 0, expect = 0;
for (size_t i = 0; i < opcode_bitsize; i++) {
const size_t bit_position = opcode_bitsize - i - 1;
switch (bitstring[i]) {
case '0':
mask |= 1 << bit_position;
mask |= one << bit_position;
break;
case '1':
expect |= 1 << bit_position;
mask |= 1 << bit_position;
expect |= one << bit_position;
mask |= one << bit_position;
break;
default:
// Ignore
@@ -62,6 +63,7 @@ private:
*/
template<size_t N>
static auto GetArgInfo(const char* const bitstring) {
const auto one = static_cast<opcode_type>(1);
std::array<opcode_type, N> masks = {};
std::array<size_t, N> shifts = {};
size_t arg_index = 0;
@@ -84,7 +86,7 @@ private:
}
ASSERT(arg_index < N);
masks[arg_index] |= 1 << bit_position;
masks[arg_index] |= one << bit_position;
shifts[arg_index] = bit_position;
}
}