diff options
author | Christophe Lyon <christophe.lyon@linaro.org> | 2015-01-20 16:04:24 +0100 |
---|---|---|
committer | Christophe Lyon <christophe.lyon@linaro.org> | 2015-01-20 16:04:24 +0100 |
commit | c1cc7826d74587e0dc1c855810633a219b161ab3 (patch) | |
tree | 8aa68e368da3d9ebfa0a6c4feee1ca50c721dd2c /ref_vqdmlal_lane.c | |
parent | 4b5570c6f0b3f1465db686a09ad1b6356df2676e (diff) | |
download | platform_external_arm-neon-tests-c1cc7826d74587e0dc1c855810633a219b161ab3.tar.gz platform_external_arm-neon-tests-c1cc7826d74587e0dc1c855810633a219b161ab3.tar.bz2 platform_external_arm-neon-tests-c1cc7826d74587e0dc1c855810633a219b161ab3.zip |
__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.
Diffstat (limited to 'ref_vqdmlal_lane.c')
-rw-r--r-- | ref_vqdmlal_lane.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ref_vqdmlal_lane.c b/ref_vqdmlal_lane.c index 53073d8..e7d42f7 100644 --- a/ref_vqdmlal_lane.c +++ b/ref_vqdmlal_lane.c @@ -44,7 +44,7 @@ FNNAME (INSN_NAME) /* vector_res = vqdmlxl_lane(vector, vector3, vector4, lane), then store the result. */ #define TEST_VQDMLXL_LANE1(INSN, T1, T2, W, W2, N, V) \ - 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##_##T2##W2(VECT_VAR(vector, T1, W, N), \ VECT_VAR(vector3, T1, W2, N), \ |