Skip to content

RISC-V: fix hardcoded options in RVV 0.7.1 toolchain file#23418

Merged
asmorkalov merged 1 commit intoopencv:4.xfrom
mshabunin:fix-rvv-07-old
Apr 2, 2023
Merged

RISC-V: fix hardcoded options in RVV 0.7.1 toolchain file#23418
asmorkalov merged 1 commit intoopencv:4.xfrom
mshabunin:fix-rvv-07-old

Conversation

@mshabunin
Copy link
Copy Markdown
Contributor

@mshabunin mshabunin commented Mar 27, 2023

  • CMAKE_*_FLAGS contained excessive compiler flags
  • CMAKE_SYSTEM_PROCESSOR was missing
  • Used CMAKE_*_FLAGS_INIT (cmake 3.7+)

Tested with the following configuration:

PATH=/opt/riscv64-linux-x86_64-20210618/bin:${PATH} \
cmake \
    -DCMAKE_BUILD_TYPE=Release \
    -DCMAKE_TOOLCHAIN_FILE=/opencv/platforms/linux/riscv64-071-gcc.toolchain.cmake \
    -DBUILD_SHARED_LIBS=OFF \
    /opencv

@asmorkalov asmorkalov self-requested a review April 2, 2023 09:43
@asmorkalov asmorkalov self-assigned this Apr 2, 2023
@asmorkalov asmorkalov merged commit 3ac06bf into opencv:4.x Apr 2, 2023
@mshabunin mshabunin deleted the fix-rvv-07-old branch April 3, 2023 10:54
@asmorkalov asmorkalov mentioned this pull request May 31, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@asmorkalov asmorkalov asmorkalov approved these changes

Assignees

@asmorkalov asmorkalov

Labels

category: build/install platform: riscv

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mshabunin @asmorkalov