[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-20 Thread Craig Topper 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 rGa29dfc089d18: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics. (authored by craig.topper). Changed prior to commit: https://review

[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-20 Thread Zakk Chen via Phabricator via cfe-commits
khchen accepted this revision. khchen added a comment. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D112020/new/ https://reviews.llvm.org/D112020 ___ cfe-commits mailing list cfe-commits@lists.llvm

[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-20 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai accepted this revision. HsiangKai added a comment. This revision is now accepted and ready to land. I ran tests under `clang/test/CodeGen/RISCV/rvv-intrinsics/`. It spent a little more time to run the tests. (previous: 84.01s; apply this patch: 96.43s in my local environment.) We could

[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-20 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai added a comment. In D112020#3074557 , @HsiangKai wrote: > After preprocessing, macro definitions will disappear, but function > declarations will not. This is the benefit of using macro. > > After applying D112102

[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-20 Thread Hsiangkai Wang via Phabricator via cfe-commits
HsiangKai added a comment. After preprocessing, macro definitions will disappear, but function declarations will not. This is the benefit of using macro. After applying D112102 , we could still use #define vadd_vv_i8m1(op0, op1, op2) \ __builtin_rvv_vadd_vv

[PATCH] D112020: [RISCV] Use clang_builtin_alias for all RISCV vector intrinsics.

2021-10-18 Thread Craig Topper via Phabricator via cfe-commits
craig.topper created this revision. craig.topper added reviewers: khchen, arcbbb, kito-cheng, HsiangKai, evandro. Herald added subscribers: achieveartificialintelligence, jeroen.dobbelaere, StephenFan, vkmr, frasercrmck, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei,