github-actions[bot]
commented
3 hours ago Lint Status
The following issues have been found with 40725-RFCRISCV_Add_target_dependent_pass_to_optimize_related_permutation_constants-1 using gcc's ./contrib/check_GNU_style.py. Please use your best judgement when resolving these issues. These are only warnings and do not need to be resolved in order to merge your patch.
=== ERROR type #1: braces should be on a separate line (1 error(s)) ===
gcc/config/riscv/riscv-vect-permconst.cc:138:52: : rtl_opt_pass (pass_data_vect_permconst, ctxt) {}
=== ERROR type #2: dot, space, space, end of comment (9 error(s)) ===
gcc/config/riscv/riscv-vect-permconst.cc:53:20: RTL_PASS, /* type */
gcc/config/riscv/riscv-vect-permconst.cc:54:28: "vect_permconst", /* name */
gcc/config/riscv/riscv-vect-permconst.cc:55:33: OPTGROUP_NONE, /* optinfo_flags */
gcc/config/riscv/riscv-vect-permconst.cc:56:20: TV_NONE, /* tv_id */
gcc/config/riscv/riscv-vect-permconst.cc:57:29: 0, /* properties_required */
gcc/config/riscv/riscv-vect-permconst.cc:58:29: 0, /* properties_provided */
gcc/config/riscv/riscv-vect-permconst.cc:59:30: 0, /* properties_destroyed */
gcc/config/riscv/riscv-vect-permconst.cc:60:26: 0, /* todo_flags_start */
gcc/config/riscv/riscv-vect-permconst.cc:61:27: 0, /* todo_flags_finish */
=== ERROR type #3: lines should not exceed 80 characters (1 error(s)) ===
gcc/config.gcc:552:80: extra_objs="${extra_objs} riscv-v.o riscv-vsetvl.o riscv-vector-costs.o riscv-avlprop.o riscv-vect-permconst.o"
=== ERROR type #4: there should be exactly one space between function name and parenthesis (1 error(s)) ===
gcc/config/riscv/riscv-vect-permconst.cc:7:50:the Free Software Foundation; either version 3, or(at your option)
Precommit CI Run information
Logs can be found in the associated Github Actions run: https://github.com/ewlu/gcc-precommit-ci/actions/runs/11846184095
Patch information
Applied patches: 1 -> 1 Associated series: https://patchwork.sourceware.org/project/gcc/list/?series=40725 Last patch applied: https://patchwork.sourceware.org/project/gcc/patch/88962370-f9c6-4593-93f1-a6b5b5dff844@ventanamicro.com/ Patch id: 101145
Build Targets
multilib, please refer to the table below to see all the targets within that multilib.-marchstringrv64gcv-lp64d,rv32gc-ilp32d,rv64gc-lp64d,rv32imc_zba_zbb_zbc_zbs-ilp32rv32gcv-ilp32d,rv64gcv-lp64drv32gc_zba_zbb_zbc_zbs-ilp32d,rv64gc_zba_zbb_zbc_zbs-lp64dTarget Information
-marchstringgc_zba_zbb_zbc_zbsNotes
Testsuite results use a more lenient allowlist to reduce error reporting with flakey tests. Please take a look at the current allowlist. Results come from a sum file comparator. Each patch is applied to a well known, non-broken baseline taken from our gcc postcommit framework (here) which runs the full gcc testsuite every 6 hours. If you have any questions or encounter any issues which may seem like false-positives, please contact us at patchworks-ci@rivosinc.com