Closed github-actions[bot] closed 11 months ago
The following issues have been found with 26034-RISCV_Add_AVL_propagation_PASS_for_RVV_autovectorization-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. If any of these warnings seem like false-positives that could be guarded against please contact me: patchworks-ci@rivosinc.com.
=== ERROR type #1: braces should be on a separate line (1 error(s)) ===
gcc/config/riscv/riscv-avlprop.cc:104:77: pass_avlprop (gcc::context *ctxt) : rtl_opt_pass (pass_data_avlprop, ctxt) {}
=== ERROR type #2: dot, space, space, end of comment (12 error(s)) ===
gcc/config/riscv/riscv-avlprop.cc:90:20: RTL_PASS, /* type */
gcc/config/riscv/riscv-avlprop.cc:91:21: "avlprop", /* name */
gcc/config/riscv/riscv-avlprop.cc:92:33: OPTGROUP_NONE, /* optinfo_flags */
gcc/config/riscv/riscv-avlprop.cc:93:20: TV_NONE, /* tv_id */
gcc/config/riscv/riscv-avlprop.cc:94:29: 0, /* properties_required */
gcc/config/riscv/riscv-avlprop.cc:95:29: 0, /* properties_provided */
gcc/config/riscv/riscv-avlprop.cc:96:30: 0, /* properties_destroyed */
gcc/config/riscv/riscv-avlprop.cc:97:26: 0, /* todo_flags_start */
gcc/config/riscv/riscv-avlprop.cc:98:27: 0, /* todo_flags_finish */
gcc/config/riscv/riscv-avlprop.cc:282:77: /* Go through all the instructions looking for AVL that we could propagate.█*/
gcc/config/riscv/riscv-v.cc:4451:77: to emit vsetvl. We pick up the default value according to prefer policy.█*/
gcc/config/riscv/riscv-v.cc:4497:43: all RVV instructions that have VTYPE OP.█*/
=== ERROR type #3: dot, space, space, new sentence (7 error(s)) ===
gcc/config/riscv/riscv-avlprop.cc:26:49: there will become even harder to maintain.█A straight forward
gcc/config/riscv/riscv-avlprop.cc:167:34: arithmetic operations (e.d.█DIV), then do the whole vector register
gcc/config/riscv/riscv-avlprop.cc:173:65: generates the VLMAX instruction due to missed LEN information.█The later
gcc/config/riscv/riscv-avlprop.cc:198:18: instruction.█It should be totally enough for vectorized codes since
gcc/config/riscv/riscv-v.cc:4451:20: to emit vsetvl.█We pick up the default value according to prefer policy. */
gcc/config/riscv/riscv-v.cc:4488:38:/* Helper function to get SEW operand.█We always have SEW value for
gcc/config/riscv/riscv-v.cc:4496:40:/* Helper function to get VLMUL operand.█We always have VLMUL value for
=== ERROR type #4: lines should not exceed 80 characters (1 error(s)) ===
gcc/config.gcc:547:80: extra_objs="${extra_objs} riscv-v.o riscv-vsetvl.o riscv-vector-costs.o riscv-avlprop.o"
=== ERROR type #5: there should be exactly one space between function name and parenthesis (1 error(s)) ===
gcc/config/riscv/riscv-avlprop.cc:9:50:the Free Software Foundation; either version 3, or(at your option)
=== ERROR type #6: trailing whitespace (1 error(s)) ===
gcc/config/riscv/t-riscv:74:65: $(TARGET_H) tree-pass.h df.h rtl-ssa.h cfgcleanup.h insn-attr.h█
Target | Status |
---|---|
Baseline hash: https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 | Applied |
Tip of tree hash: https://github.com/gcc-mirror/gcc/commit/2287fa29fd28ae39963de80d5f845be2c6ab8675 | Applied |
Patch applied successfully
Target | Status |
---|---|
linux-rv64gcv-lp64d-multilib | Success |
newlib-rv64gc-lp64d-multilib | Success |
linux-rv64gc_zba_zbb_zbc_zbs-lp64d-non-multilib | Success |
linux-rv32gc_zba_zbb_zbc_zbs-ilp32d-non-multilib | Success |
Patch(es) were applied to the baseline hash https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456. If this patch commit depends on or conflicts with a recently committed patch, then these results may be outdated.
New Failures | gcc | g++ | gfortran | Previous Hash |
---|---|---|---|---|
linux: rv32gcv ilp32d medlow multilib | 37/12 | 4/1 | 31/11 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
Resolved Failures | gcc | g++ | gfortran | Previous Hash |
---|---|---|---|---|
linux: rv32gcv ilp32d medlow multilib | 0/0 | 0/0 | 1/1 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
linux: rv64gcv lp64d medlow multilib | 0/0 | 0/0 | 1/1 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
Unresolved Failures | gcc | g++ | gfortran | Previous Hash |
---|---|---|---|---|
linux: rv32 Bitmanip ilp32d medlow | 32/15 | 17/7 | 30/5 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
linux: rv32gcv ilp32d medlow multilib | 233/79 | 29/17 | 31/14 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
linux: rv64 Bitmanip lp64d medlow | 34/17 | 13/4 | 30/5 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
linux: rv64gcv lp64d medlow multilib | 106/55 | 13/4 | 30/11 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
newlib: rv32gc ilp32d medlow multilib | 81/22 | 107/15 | 0/0 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
newlib: rv64gc lp64d medlow multilib | 74/21 | 79/10 | 0/0 | https://github.com/gcc-mirror/gcc/commit/9cf2e7441eeeb707cc393d6d211184df8f56d456 |
FAIL: g++.dg/vect/pr110557.cc -std=c++14 execution test
FAIL: g++.dg/vect/pr110557.cc -std=c++17 execution test
FAIL: g++.dg/vect/pr110557.cc -std=c++20 execution test
FAIL: g++.dg/vect/pr110557.cc -std=c++98 execution test
FAIL: gcc.dg/nextafter-2.c execution test
FAIL: gcc.dg/vect/vect-alias-check-16.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-alias-check-16.c execution test
FAIL: gcc.dg/vect/vect-alias-check-20.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-alias-check-20.c execution test
FAIL: gcc.dg/vect/vect-reduc-10.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-reduc-10.c execution test
FAIL: gcc.dg/vect/vect-reduc-dot-19.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-reduc-dot-19.c execution test
FAIL: gcc.dg/vect/vect-reduc-dot-20.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-reduc-dot-20.c execution test
FAIL: gcc.dg/vect/vect-reduc-dot-21.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-reduc-dot-21.c execution test
FAIL: gcc.dg/vect/vect-reduc-dot-22.c -flto -ffat-lto-objects execution test
FAIL: gcc.dg/vect/vect-reduc-dot-22.c execution test
FAIL: gcc.target/riscv/rvv/autovec/reduc/extract_last_run-10.c execution test
FAIL: gcc.target/riscv/rvv/autovec/reduc/extract_last_run-9.c execution test
FAIL: gcc.target/riscv/rvv/autovec/reduc/reduc_run-1.c execution test
FAIL: gcc.target/riscv/rvv/autovec/unop/popcount-run-1.c execution test
FAIL: gfortran.dg/default_format_2.f90 -O0 execution test
FAIL: gfortran.dg/default_format_2.f90 -O1 execution test
FAIL: gfortran.dg/default_format_2.f90 -O2 execution test
FAIL: gfortran.dg/default_format_2.f90 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions execution test
FAIL: gfortran.dg/default_format_2.f90 -O3 -g execution test
FAIL: gfortran.dg/default_format_2.f90 -Os execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -O0 execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -O1 execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -O2 execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -O3 -g execution test
FAIL: gfortran.dg/default_format_denormal_2.f90 -Os execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -O0 execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -O1 execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -O2 execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -O3 -g execution test
FAIL: gfortran.dg/ieee/dec_math_1.f90 -Os execution test
FAIL: gfortran.dg/ieee/large_1.f90 -O0 execution test
FAIL: gfortran.dg/ieee/large_2.f90 -O0 execution test
FAIL: gfortran.dg/ieee/large_2.f90 -O1 execution test
FAIL: gfortran.dg/ieee/large_2.f90 -O2 execution test
FAIL: gfortran.dg/ieee/large_2.f90 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions execution test
FAIL: gfortran.dg/ieee/large_2.f90 -O3 -g execution test
FAIL: gfortran.dg/ieee/large_2.f90 -Os execution test
FAIL: gfortran.dg/large_real_kind_2.F90 -O0 execution test
FAIL: gfortran.dg/round_4.f90 -O0 execution test
FAIL: gfortran.dg/zero_sized_3.f90 -O3 -fomit-frame-pointer -funroll-loops -fpeel-loops -ftracer -finline-functions execution test
FAIL: gfortran.fortran-torture/execute/intrinsic_count.f90 execution, -O2 -fomit-frame-pointer -finline-functions -funroll-loops
FAIL: gfortran.fortran-torture/execute/intrinsic_matmul.f90 execution, -O2 -fomit-frame-pointer -finline-functions -funroll-loops
FAIL: gfortran.fortran-torture/execute/intrinsic_sum.f90 execution, -O2 -fomit-frame-pointer -finline-functions -funroll-loops
linux rv32gcv ilp32d medlow multilib:
FAIL: gfortran.dg/matmul_3.f90 -O2 execution test
linux rv64gcv lp64d medlow multilib:
FAIL: gfortran.dg/matmul_6.f90 -O2 execution test
Precommit CI Run information
Logs can be found in the associated Github Actions run: https://github.com/ewlu/gcc-precommit-ci/actions/runs/6628017319
Patch information
Applied patches: 1 -> 1 Associated series: https://patchwork.sourceware.org/project/gcc/list/?series=26034 Last patch applied: https://patchwork.sourceware.org/project/gcc/patch/20231024033200.224558-1-juzhe.zhong@rivai.ai/
Notes
Testsuite checks are in beta testing stages. Results are unstable and may be inaccurate.