aboutsummaryrefslogtreecommitdiff
path: root/gcc/genhooks.c
diff options
context:
space:
mode:
authorKyrylo Tkachov <kyrylo.tkachov@arm.com>2021-02-01 22:51:11 +0000
committerKyrylo Tkachov <kyrylo.tkachov@arm.com>2021-02-02 12:14:39 +0000
commite8062ad468062f40064d2a648f02f993fe0070ba (patch)
tree906887adfe634725f14bf56c88bf1034acc308d5 /gcc/genhooks.c
parent5cebc818213c1281d54f3d1d1676674f995241fa (diff)
downloadgcc-e8062ad468062f40064d2a648f02f993fe0070ba.zip
gcc-e8062ad468062f40064d2a648f02f993fe0070ba.tar.gz
gcc-e8062ad468062f40064d2a648f02f993fe0070ba.tar.bz2
aarch64: Add and use FLAG_LOAD in builtins
We already have a STORE flag that we use for builtins. This patch introduces a LOAD set that uses AUTO_FP and FLAG_READ_MEMORY. This allows for more aggressive optimisation of the load intrinsics. Turns out we have a great many testcases that do: float16x4x2_t f_vld2_lane_f16 (float16_t * p, float16x4x2_t v) { float16x4x2_t res; /* { dg-error "lane 4 out of range 0 - 3" "" { target *-*-* } 0 } */ res = vld2_lane_f16 (p, v, 4); /* { dg-error "lane -1 out of range 0 - 3" "" { target *-*-* } 0 } */ res = vld2_lane_f16 (p, v, -1); return res; } but since the first res is unused it now gets eliminated early on before we get to give an error message. Ideally we'd like to warn for both. This patch takes the conservative approach and doesn't convert the load-lane builtins to LOAD ; that's something we can improve later. gcc/ChangeLog: * config/aarch64/aarch64-builtins.c (FLAG_LOAD): Define. * config/aarch64/aarch64-simd-builtins.def (ld1x2, ld2, ld3, ld4, ld2r, ld3r, ld4r, ld1, ld1x3, ld1x4): Use LOAD flags.
Diffstat (limited to 'gcc/genhooks.c')
0 files changed, 0 insertions, 0 deletions