summaryrefslogtreecommitdiff
path: root/Source/Core/InputCommon/ControllerInterface
diff options
context:
space:
mode:
authorLioncash <mathew1800@gmail.com>2018-12-14 13:23:12 -0500
committerLioncash <mathew1800@gmail.com>2018-12-25 10:35:09 -0500
commit244d083f0e2e0084fb5b597db4ffd0884b0e7495 (patch)
treeff602005c70a24eb6110b49d7c9099618039e08a /Source/Core/InputCommon/ControllerInterface
parent2dcd058f7d87a0a95289b23473ae0e7321005110 (diff)
PowerPC: Remove separate macros for paired singles
Previously, PowerPC.h had four macros in it like so: \#define rPS0(i) (*(double*)(&PowerPC::ppcState.ps[i][0])) \#define rPS1(i) (*(double*)(&PowerPC::ppcState.ps[i][1])) \#define riPS0(i) (*(u64*)(&PowerPC::ppcState.ps[i][0])) \#define riPS1(i) (*(u64*)(&PowerPC::ppcState.ps[i][1])) Casting between object representations like this is undefined behavior. Given this is used heavily with the interpreter (that is, the most accurate, but slowest CPU backend), we don't exactly want to allow undefined behavior to creep into it. Instead, this adds a helper struct for operating with the paired singles, and replaces the four macros with a single macro for accessing the paired-singles/floating-point registers. This way, it's left up to the caller to explicitly decide how it wants to interpret the data (and makes it more obvious where different interpretations of the same data are occurring at, as there'll be a call to one of the [x]AsDouble() functions).
Diffstat (limited to 'Source/Core/InputCommon/ControllerInterface')
0 files changed, 0 insertions, 0 deletions