Skip to content

RISC-V: fix build with RVV 0.7.1#26266

Merged
asmorkalov merged 1 commit intoopencv:4.xfrom
mshabunin:fix-rvv071-build
Oct 7, 2024
Merged

RISC-V: fix build with RVV 0.7.1#26266
asmorkalov merged 1 commit intoopencv:4.xfrom
mshabunin:fix-rvv071-build

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Oct 7, 2024

Issue introduced in #25891 (https://github.com/opencv/ci-gha-workflow/actions/runs/11188809334/job/31165423898)

/opencv/modules/core/include/opencv2/core/hal/intrin_rvv071.hpp:360:36: error: template-id 'v_setall_<>' for 'cv::hal_baseline::v_int8x16 cv::hal_baseline::v_setall_(char)' does not match any template declaration
  360 | template <> inline v_##_Tp##x##num v_setall_(__Tp v) { return v_setall_##suffix(v); }
      |                                    ^~~~~~~~~
/opencv/modules/core/include/opencv2/core/hal/intrin_rvv071.hpp:363:1: note: in expansion of macro 'OPENCV_HAL_IMPL_RISCVV_INIT_SET'
  363 | OPENCV_HAL_IMPL_RISCVV_INIT_SET(char, int8, s8, i8, 16)
      | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from /opencv/modules/core/src/precomp.hpp:88,
                 from /opencv/modules/core/src/array.cpp:49:
/opencv/modules/core/include/opencv2/core/hal/intrin.hpp:205:42: note: candidates are: 'template<class _VecTp> _VecTp cv::hal_baseline::v_setall_(double)'
  205 | template <typename _VecTp> inline _VecTp v_setall_(double);
      |                                          ^~~~~~~~~
/opencv/modules/core/include/opencv2/core/hal/intrin.hpp:204:42: note:                 'template<class _VecTp> _VecTp cv::hal_baseline::v_setall_(float)'
  204 | template <typename _VecTp> inline _VecTp v_setall_(float);
      |                                          ^~~~~~~~~
...

@asmorkalov asmorkalov merged commit cda9f41 into opencv:4.x Oct 7, 2024
@mshabunin mshabunin deleted the fix-rvv071-build branch October 7, 2024 19:40
@asmorkalov asmorkalov mentioned this pull request Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants