[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

fhahn added a comment. Herald added a subscriber: pcwang-thead. Herald added a project: All. Unfortunately some of the intrinsics tests have grown so large the the consistently hit the 10 minute timeout in the Phabricator precommit tests, e.g. see https://reviews.llvm.org/harbormaster/unit/15222

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

fhahn added a comment. Also see https://github.com/llvm/llvm-project/issues/54168 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117860/new/ https://reviews.llvm.org/D117860 ___ cfe-commits mailing list c

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

This revision was automatically updated to reflect the committed changes. Closed by commit rG3cf15af2daa9: [RISCV] Remove experimental prefix from rvv-related extensions. (authored by eopXD). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117860/new/

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117860/new/ https://reviews.llvm.org/D117860 ___

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

eopXD updated this revision to Diff 402192. eopXD added a comment. Rebase. Resolve conflict due to addition of zbkc. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117860/new/ https://reviews.llvm.org/D117860 Files: clang/test/CodeGen/RISCV/riscv

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

eopXD added a comment. In D117860#3261453 , @frasercrmck wrote: > Thanks for the patch! Not sure the best way to review this. The tests are > passing, which is a good sign. One option could be to split the "meaningful" > changes into a separate diff fo

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

eopXD updated this revision to Diff 402034. eopXD added a comment. Cleanup redundant code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117860/new/ https://reviews.llvm.org/D117860 Files: clang/test/CodeGen/RISCV/riscv-attr-builtin-alias-err.c

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

craig.topper added inline comments. Comment at: llvm/lib/Support/RISCVISAInfo.cpp:429 StringRef ExtName = Feature; +dbgs() << ExtName << "\n"; bool Experimental = false; Drop this? Or wrap it in LLVM_DEBUG Repository: rG LLVM Github Monorepo CH

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

frasercrmck added a comment. Thanks for the patch! Not sure the best way to review this. The tests are passing, which is a good sign. One option could be to split the "meaningful" changes into a separate diff for easier viewing? We probably don't need to see all the test RUN line changes, for e

[PATCH] D117860: [RISCV] Remove experimental prefix from rvv-related extensions.

eopXD created this revision. eopXD added reviewers: craig.topper, asb, frasercrmck. Herald added subscribers: VincentWu, luke957, achieveartificialintelligence, armkevincheng, eric-k256, vkmr, jdoerfert, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, a