From c1cc7826d74587e0dc1c855810633a219b161ab3 Mon Sep 17 00:00:00 2001 From: Christophe Lyon Date: Tue, 20 Jan 2015 16:04:24 +0100 Subject: __set_neon_cumulative_sat() modifies the contents on the QC flag, and some intrinsics do so too: this patch adds the explicit dependency on the asm statement, to avoid code reordering or removal. When writing QC, the asm statement now has a fake input dependency, which is the output of the intrinsic being tested. Modifying the __set_neon_cumulative_sat macro is necessary, to be able to accept all the possible input types. Update the generic code in ref_v_binary_sat_op.c and ref_v_unary_sat_op.c accordingly, as well as all the tests involving QC. --- ref_vqrshl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'ref_vqrshl.c') diff --git a/ref_vqrshl.c b/ref_vqrshl.c index 5028bf9..e4a33e5 100644 --- a/ref_vqrshl.c +++ b/ref_vqrshl.c @@ -41,7 +41,7 @@ FNNAME (INSN) { /* Basic test: v3=vqrshl(v1,v2), then store the result. */ #define TEST_VQRSHL2(INSN, T3, Q, T1, T2, W, N) \ - Set_Neon_Cumulative_Sat(0); \ + Set_Neon_Cumulative_Sat(0, VECT_VAR(vector_res, T1, W, N)); \ VECT_VAR(vector_res, T1, W, N) = \ INSN##Q##_##T2##W(VECT_VAR(vector, T1, W, N), \ VECT_VAR(vector_shift, T3, W, N)); \ -- cgit v1.2.3