[gcc r14-1789] RISC-V: Add comments of some functions

Pan Li panli@gcc.gnu.org
Tue Jun 13 14:14:29 GMT 2023


https://gcc.gnu.org/g:a07dadba85f1b15e270c227dfa70e2fdf331494f

commit r14-1789-ga07dadba85f1b15e270c227dfa70e2fdf331494f
Author: Juzhe-Zhong <juzhe.zhong@rivai.ai>
Date:   Tue Jun 13 10:26:11 2023 +0800

    RISC-V: Add comments of some functions
    
    gcc/ChangeLog:
    
            * config/riscv/riscv-v.cc (rvv_builder::single_step_npatterns_p): Add comment.
            (shuffle_generic_patterns): Ditto.
            (expand_vec_perm_const_1): Ditto.

Diff:
---
 gcc/config/riscv/riscv-v.cc | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/gcc/config/riscv/riscv-v.cc b/gcc/config/riscv/riscv-v.cc
index 34fdb53c140..d797326d736 100644
--- a/gcc/config/riscv/riscv-v.cc
+++ b/gcc/config/riscv/riscv-v.cc
@@ -457,6 +457,9 @@ rvv_builder::single_step_npatterns_p () const
      { 2, 2, 2, 2, 4, 4, 4, 4, 8, 8, 8, 8, 16, 16, 16, 16, ... }
    E.g. NPATTERNS = 8:
      { 2, 2, 2, 2, 2, 2, 2, 2, 8, 8, 8, 8, 8, 8, 8, 8, ... }
+   We only check ele[0] ~ ele[NPATTERNS - 1] whether they are the same.
+   We don't need to check the elements[n] with n >= NPATTERNS since
+   they don't belong to the same pattern.
 */
 bool
 rvv_builder::npatterns_all_equal_p () const
@@ -2478,6 +2481,8 @@ shuffle_generic_patterns (struct expand_vec_perm_d *d)
   return true;
 }
 
+/* This function recognizes and supports different permutation patterns
+   and enable VLA SLP auto-vectorization.  */
 static bool
 expand_vec_perm_const_1 (struct expand_vec_perm_d *d)
 {
@@ -2509,6 +2514,8 @@ expand_vec_perm_const_1 (struct expand_vec_perm_d *d)
   return false;
 }
 
+/* This function implements TARGET_VECTORIZE_VEC_PERM_CONST by using RVV
+ * instructions.  */
 bool
 expand_vec_perm_const (machine_mode vmode, machine_mode op_mode, rtx target,
 		       rtx op0, rtx op1, const vec_perm_indices &sel)


More information about the Gcc-cvs mailing list