diff options
author | Pan Li <pan2.li@intel.com> | 2024-05-11 15:25:28 +0800 |
---|---|---|
committer | Pan Li <pan2.li@intel.com> | 2024-05-13 22:58:40 +0800 |
commit | 41b3cf262e61aee9d26380f1c820e0eaae740f50 (patch) | |
tree | d887460d0cbf547ea63d683478781c872df6201c /gcc/expr.cc | |
parent | 6d31a370e26eeb950c326332633b3e8e84b6630b (diff) | |
download | gcc-41b3cf262e61aee9d26380f1c820e0eaae740f50.zip gcc-41b3cf262e61aee9d26380f1c820e0eaae740f50.tar.gz gcc-41b3cf262e61aee9d26380f1c820e0eaae740f50.tar.bz2 |
RISC-V: Bugfix ICE for RVV intrinisc vfw on _Float16 scalar
For the vfw vx format RVV intrinsic, the scalar type _Float16 also
requires the zvfh extension. Unfortunately, we only check the
vector tree type and miss the scalar _Float16 type checking. For
example:
vfloat32mf2_t test_vfwsub_wf_f32mf2(vfloat32mf2_t vs2, _Float16 rs1, size_t vl)
{
return __riscv_vfwsub_wf_f32mf2(vs2, rs1, vl);
}
It should report some error message like zvfh extension is required
instead of ICE for unreg insn.
This patch would like to make up such kind of validation for _Float16
in the RVV intrinsic API. It will report some error like below when
there is no zvfh enabled.
error: built-in function '__riscv_vfwsub_wf_f32mf2(vs2, rs1, vl)'
requires the zvfhmin or zvfh ISA extension
Passed the rv64gcv fully regression tests, included c/c++/fortran.
PR target/114988
gcc/ChangeLog:
* config/riscv/riscv-vector-builtins.cc
(validate_instance_type_required_extensions): New func impl to
validate the intrinisc func type ops.
(expand_builtin): Validate instance type before expand.
gcc/testsuite/ChangeLog:
* gcc.target/riscv/rvv/base/pr114988-1.c: New test.
* gcc.target/riscv/rvv/base/pr114988-2.c: New test.
Signed-off-by: Pan Li <pan2.li@intel.com>
Diffstat (limited to 'gcc/expr.cc')
0 files changed, 0 insertions, 0 deletions