Skip to content

riscv: tidy the RVV feature gate in compiler.h - #4893

Open
ihb2032 wants to merge 1 commit into
meta:devfrom
ihb2032:riscv-rvv-gate-tidy
Open

ihb2032 wants to merge 1 commit into
meta:devfrom
ihb2032:riscv-rvv-gate-tidy

Conversation

@ihb2032

@ihb2032 ihb2032 commented Nov 5, 2026

Copy link
Copy Markdown

What and why

  • Document why the RVV gate requires GCC >= 14 / Clang >= 19 (first releases with the stable RVV 1.0 intrinsic API).
  • Fix the preprocessor nesting of the version check and drop a stray # between the arch blocks.

Checklist

  • I searched open pull requests and issues: this problem is not already being addressed.
  • Bug fix: it comes with a reproducer or a test that fails without the fix.
  • Performance change: numbers from repeated runs are included, with CPU, compiler, flags and data.
    A gain in one configuration is not a general gain: see the Performance section of CONTRIBUTING.md.
  • I stated below whether AI or other automated tools were used.

Tools used

None

Co-authored-by: lyd1992 <liuyudong@iscas.ac.cn>
@meta-cla meta-cla Bot added the CLA Signed label Nov 5, 2026

This branch has not been deployed

No deployments
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.

1 participant