summaryrefslogtreecommitdiffstatshomepage
diff options
context:
space:
mode:
author Alex W. Jackson <alex.w.jackson@gmail.com>2015-06-26 12:45:18 -0400
committer Alex W. Jackson <alex.w.jackson@gmail.com>2015-06-26 12:45:41 -0400
commit91be2c989ee5d2d028cab770920b54803ea2cd41 (patch)
tree4afba70a0111bfb18f03d429992fc237c5f1580a
parent382dd0f616b7e54a20885d68d8ff981dc4197109 (diff)
validity: add basic signed integer arithmetic tests to core validation [Alex Jackson]
-rw-r--r--src/emu/validity.c12
1 files changed, 12 insertions, 0 deletions
diff --git a/src/emu/validity.c b/src/emu/validity.c
index 14f46661888..fe4c14cfa8c 100644
--- a/src/emu/validity.c
+++ b/src/emu/validity.c
@@ -326,6 +326,8 @@ void validity_checker::validate_one(const game_driver &driver)
void validity_checker::validate_core()
{
// basic system checks
+ if (~0 != -1) osd_printf_error("Machine must be two's complement\n");
+
UINT8 a = 0xff;
UINT8 b = a + 1;
if (b > a) osd_printf_error("UINT8 must be 8 bits\n");
@@ -340,6 +342,16 @@ void validity_checker::validate_core()
if (sizeof(INT64) != 8) osd_printf_error("INT64 must be 64 bits\n");
if (sizeof(UINT64) != 8) osd_printf_error("UINT64 must be 64 bits\n");
+ // check signed right shift
+ INT8 a8 = -3;
+ INT16 a16 = -3;
+ INT32 a32 = -3;
+ INT64 a64 = -3;
+ if (a8 >> 1 != -2) osd_printf_error("INT8 right shift must be arithmetic\n");
+ if (a16 >> 1 != -2) osd_printf_error("INT16 right shift must be arithmetic\n");
+ if (a32 >> 1 != -2) osd_printf_error("INT32 right shift must be arithmetic\n");
+ if (a64 >> 1 != -2) osd_printf_error("INT64 right shift must be arithmetic\n");
+
// check pointer size
#ifdef PTR64
if (sizeof(void *) != 8) osd_printf_error("PTR64 flag enabled, but was compiled for 32-bit target\n");