Skip to content

Conversation

@cbalint13
Copy link
Contributor

@cbalint13 cbalint13 commented Apr 18, 2025

This PR enables VLA (Vector-Length-Agnostic / Variable-Length-Array) predication for RISCV targets.


New functionality:

  • Enables existing SVE as generic VLA predication, extending the support done for aarch64 to riscv{32,64}

Changes:

  • Vscale rage is now computed from arch specific vector_width (see disscuss), adapting to any vector ISA
  • Split SVE related test cases between arch specific {riscv, aarch64} and llvm VLA generic (common) ones

UPDATE:

  • Changed terms to generic VLA for common or non AArch64 parts (SVE remains ARM specific, RVV is riscv specific).

@cbalint13 cbalint13 force-pushed the riscv-sve branch 8 times, most recently from ea1b57b to 4bbdd8c Compare April 19, 2025 11:29
@cbalint13
Copy link
Contributor Author

This is ready to review.

Cc: @lhutton1, @ekalda
Cc: @quic-sanirudh , @srkreddy1238
Cc: @Hzfengsy , @spectrometerHBH , @tqchen

Cc: @jerryzj , @JieGH

@cbalint13 cbalint13 marked this pull request as ready for review April 19, 2025 12:56
@cbalint13 cbalint13 changed the title [LLVM][Codegen] Enable SVE for RISCV targets [LLVM][Codegen] Enable SVE/VLA for RISCV targets Apr 19, 2025
@cbalint13 cbalint13 marked this pull request as draft May 12, 2025 12:31
@tqchen
Copy link
Member

tqchen commented May 12, 2025

Thanks @cbalint13 , do u mind rebase

@cbalint13 cbalint13 closed this May 13, 2025
@cbalint13 cbalint13 reopened this May 13, 2025
@cbalint13 cbalint13 force-pushed the riscv-sve branch 3 times, most recently from 9f15746 to 270f98b Compare May 13, 2025 03:58
@cbalint13 cbalint13 marked this pull request as ready for review May 13, 2025 09:19
@cbalint13
Copy link
Contributor Author

Thanks @cbalint13 , do u mind rebase

I've re-based this to the latest head.

@tqchen tqchen merged commit 01268ac into apache:main May 13, 2025
16 checks passed
@cbalint13 cbalint13 deleted the riscv-sve branch August 28, 2025 16:37
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