Skip to content

RISC-V: remove statically initialized global RVV variables (5.x)#26119

Merged
asmorkalov merged 1 commit intoopencv:5.xfrom
mshabunin:fix-rvv-init
Sep 6, 2024
Merged

RISC-V: remove statically initialized global RVV variables (5.x)#26119
asmorkalov merged 1 commit intoopencv:5.xfrom
mshabunin:fix-rvv-init

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Sep 5, 2024

Steps to reproduce:

  • RISC-V build without baseline RVV support, but with dispatched RVV code

Actual behavior:

  • Illegal instruction during library initialization

Solution:

  • avoid global variables initialized using RVV instructions

Note: should fix weekly Andes build on 5.x branch (failed testing with qemu), because it has dispatched code enabled after FP16 PR integration to 5.x

@mshabunin mshabunin changed the title RISC-V: remove statically initialized global RVV variables RISC-V: remove statically initialized global RVV variables (5.x) Sep 5, 2024
@asmorkalov asmorkalov added bug platform: riscv cleanup Code cleanup (e.g, drop legacy C-API, legacy unmaintained code) labels Sep 6, 2024
@asmorkalov asmorkalov self-assigned this Sep 6, 2024
@asmorkalov asmorkalov added this to the 5.0 milestone Sep 6, 2024
@asmorkalov asmorkalov merged commit babc669 into opencv:5.x Sep 6, 2024
@mshabunin mshabunin deleted the fix-rvv-init branch September 6, 2024 06:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug cleanup Code cleanup (e.g, drop legacy C-API, legacy unmaintained code) platform: riscv

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants