From 5d1d9ee085e6f297b4b74a26719dcf8700a125a4 Mon Sep 17 00:00:00 2001 From: Georgia Kouveli Date: Tue, 3 Oct 2017 11:32:35 +0100 Subject: Disable forward reference info negative tests in release mode. The forward reference tests require ExactAssemblyScope checks that are debug-only. Change-Id: I2847456e5f0cc68636786d0cb7a54e416fbff0c5 --- test/aarch32/test-assembler-aarch32.cc | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/test/aarch32/test-assembler-aarch32.cc b/test/aarch32/test-assembler-aarch32.cc index 6c47c6d0..eb2a40d9 100644 --- a/test/aarch32/test-assembler-aarch32.cc +++ b/test/aarch32/test-assembler-aarch32.cc @@ -6450,7 +6450,18 @@ TEST_T32(assembler_bind_label) { RUN(); } -#define TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) \ +#ifdef VIXL_DEBUG +#define TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) \ + POSITIVE_TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) \ + NEGATIVE_TEST_FORWARD_REFERENCE_INFO(INST, ASM) +#else +// Skip the negative tests for release builds, as they require debug-only checks +// in ExactAssemblyScope. +#define TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) \ + POSITIVE_TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) +#endif + +#define POSITIVE_TEST_FORWARD_REFERENCE_INFO(INST, INFO, ASM) \ can_encode = masm.INFO; \ VIXL_CHECK(can_encode); \ { \ @@ -6474,8 +6485,7 @@ TEST_T32(assembler_bind_label) { } \ Label label(pc + info->max_offset); \ masm.ASM; \ - } \ - NEGATIVE_TEST_FORWARD_REFERENCE_INFO(INST, ASM) + } #ifdef VIXL_NEGATIVE_TESTING #define NEGATIVE_TEST_FORWARD_REFERENCE_INFO(INST, ASM) \ -- cgit v1.2.3