Skip to content

feat: add RVV fast Hadamard transform operators#613

Open
ihb2032 wants to merge 2 commits into
alibaba:mainfrom
ihb2032:feat/rvv-fht-ops
Open

feat: add RVV fast Hadamard transform operators#613
ihb2032 wants to merge 2 commits into
alibaba:mainfrom
ihb2032:feat/rvv-fht-ops

Conversation

@ihb2032

@ihb2032 ihb2032 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@ihb2032
ihb2032 requested a review from iaojnh as a code owner July 23, 2026 09:12
@ihb2032 ihb2032 removed their assignment Jul 23, 2026
ihb2032 and others added 2 commits July 24, 2026 16:45
Co-authored-by: lyd1992 <liuyudong@iscas.ac.cn>
Co-authored-by: YuanSheng <yuansheng@isrc.iscas.ac.cn>
fht_rvv.cc is globbed into the build on every platform, but unlike the
other ISA files it had no arch guard and never included <riscv_vector.h>.
clang-tidy (run on x86) and non-RISC-V builds therefore failed on
undeclared RVV intrinsics/types. Wrap the body in #if defined(__riscv_vector)
and move the intrinsics header inside the guard, mirroring fht_neon.cc.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ihb2032
ihb2032 force-pushed the feat/rvv-fht-ops branch from 7192150 to 42a4abd Compare July 24, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants