[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-20 Thread Philip Reames via Phabricator via cfe-commits
reames added a comment. In D136106#3863202 , @reames wrote: > In D136106#3863147 , @craig.topper > wrote: > >>> In the original review, I'd mentioned that MinVLEN was sometimes zero. >>> That's still true, but a

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-19 Thread Philip Reames via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9a8f3b113d05: [clang][RISCV] Set vscale_range attribute based on VLEN (authored by reames). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-18 Thread Fraser Cormack via Phabricator via cfe-commits
frasercrmck accepted this revision. frasercrmck added a comment. This revision is now accepted and ready to land. LGTM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136106/new/ https://reviews.llvm.org/D136106 ___ cfe-commits mailing list cfe

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-17 Thread Philip Reames via Phabricator via cfe-commits
reames updated this revision to Diff 468344. reames edited the summary of this revision. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136106/new/ https://reviews.llvm.org/D136106 Files: clang/lib/Basic/Targets/RISCV.cpp clang/test/CodeGen/riscv-vector-bits-vscale-range.c llvm/incl

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-17 Thread Philip Reames via Phabricator via cfe-commits
reames added a comment. In D136106#3863147 , @craig.topper wrote: >> In the original review, I'd mentioned that MinVLEN was sometimes zero. >> That's still true, but apparently only happens if you specify multiple >> extensions in the -target_feature s

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-17 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. > In the original review, I'd mentioned that MinVLEN was sometimes zero. That's > still true, but apparently only happens if you specify multiple extensions in > the -target_feature string. So, we can at least test "v" and "zve64x" on > their own before I go figure

[PATCH] D136106: [clang][RISCV] Set vscale_range attribute based on VLEN

2022-10-17 Thread Philip Reames via Phabricator via cfe-commits
reames created this revision. reames added a reviewer: craig.topper. Herald added subscribers: sunshaoce, VincentWu, StephenFan, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones,