[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-28 Thread Alex Bradbury via cfe-commits
asb wrote: > > I haven't made it so if you enable all extensions that constitute Sha, Sha > > is implied. > > Great! I think it's most clear if we have "extensions only require things > they are made up of" (i.e. all the implications/requirements point to > earlier/lesser extensions) and "pro

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-28 Thread Sam Elliott via cfe-commits
lenary wrote: > I haven't made it so if you enable all extensions that constitute Sha, Sha is > implied. Great! I think it's most clear if we have "extensions only require things they are made up of" (i.e. all the implications/requirements point to earlier/lesser extensions) and "profiles are

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-28 Thread Alex Bradbury via cfe-commits
https://github.com/asb closed https://github.com/llvm/llvm-project/pull/113820 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-27 Thread Pengcheng Wang via cfe-commits
https://github.com/wangpc-pp approved this pull request. LGTM. https://github.com/llvm/llvm-project/pull/113820 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-27 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Alex Bradbury (asb) Changes This was introduced in the now-ratified RVA23 profile (and also added to the RVA22 text) as a simple way of referring to H plus the set of supervisor extensions required by RVA23. https://github.com/riscv/risc

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-27 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-backend-risc-v Author: Alex Bradbury (asb) Changes This was introduced in the now-ratified RVA23 profile (and also added to the RVA22 text) as a simple way of referring to H plus the set of supervisor extensions required by RVA23. https://github.com/r

[clang] [llvm] [RISCV] Add the Sha extension (PR #113820)

2024-10-27 Thread Alex Bradbury via cfe-commits
https://github.com/asb created https://github.com/llvm/llvm-project/pull/113820 This was introduced in the now-ratified RVA23 profile (and also added to the RVA22 text) as a simple way of referring to H plus the set of supervisor extensions required by RVA23. https://github.com/riscv/riscv-pro