Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Updated RVV intrinsics and test to remove initializer_list #25203

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Mar 14, 2024
Merged

Updated RVV intrinsics and test to remove initializer_list #25203

merged 1 commit into from
Mar 14, 2024

Conversation

@mshabunin
Copy link
Contributor

@mshabunin mshabunin commented Mar 12, 2024 •
edited by opencv-alalek
Loading

resolves #25191

force_builders=Custom
buildworker:Custom=linux-4
build_image:Custom=simd-emulator

@mshabunin mshabunin force-pushed the fix-scalable-intrin-test branch from 327c6fd to 535a656 Compare March 12, 2024 19:05
@mshabunin mshabunin mentioned this pull request Mar 12, 2024
Closed
4 tasks
opencv-alalek
opencv-alalek reviewed Mar 13, 2024
View reviewed changes
modules/core/test/test_intrin_utils.hpp Outdated
operator R () const
{
return initializer<VTraits<R>::max_nlanes>().init(*this);
CV_DbgAssert(VTraits<R>::vlanes() <= VTraits<R>::max_nlanes);
Copy link
Contributor

@opencv-alalek opencv-alalek Mar 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need to limit for debug only?

Copy link
Contributor Author

@mshabunin mshabunin Mar 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tried to make similar to removed assert. I agree, regular CV_Assert would be better, I'll fix it.

@mshabunin mshabunin force-pushed the fix-scalable-intrin-test branch from 535a656 to 6fc926e Compare March 13, 2024 18:17
@opencv-alalek opencv-alalek added this to the 4.10.0 milestone Mar 14, 2024
@opencv-alalek opencv-alalek self-assigned this Mar 14, 2024
@opencv-pushbot opencv-pushbot merged commit 625eeba into opencv:4.x Mar 14, 2024
@mshabunin mshabunin deleted the fix-scalable-intrin-test branch March 14, 2024 17:46
@asmorkalov asmorkalov mentioned this pull request Apr 1, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@opencv-alalek opencv-alalek opencv-alalek approved these changes

Assignees

@opencv-alalek opencv-alalek

Labels

platform: riscv

Projects

None yet

Milestone

4.10.0

Development

Successfully merging this pull request may close these issues.

SCALABLE RVV with debug mode encounters failing tests

3 participants

@mshabunin @opencv-alalek @opencv-pushbot