Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch Status 27773-RISCV_Rename_and_unify_stringop_strategy_handling_NFC-1 #813

Closed
github-actions bot opened this issue Dec 1, 2023 · 4 comments
Closed
Labels
linter-failure Lint failed

Comments

@github-actions
Copy link

github-actions bot commented Dec 1, 2023

Precommit CI Run information

Logs can be found in the associated Github Actions run: https://github.com/ewlu/gcc-precommit-ci/actions/runs/7062085835

Patch information

Applied patches: 1 -> 1
Associated series: https://patchwork.sourceware.org/project/gcc/list/?series=27773
Last patch applied: https://patchwork.sourceware.org/project/gcc/patch/10a0156e-6bbb-4832-9a3c-350a99f3fa46@gmail.com/
Patch id: 81120

Build Targets

Some targets are built as multilibs. If a build target ends with multilib, please refer to the table below to see all the targets within that multilib.

Target name -march string
newlib-rv64gc-lp64d-multilib rv32gc-ilp32d, rv64gc-lp64d
newlib-rv64gcv-lp64d-multilib rv64gcv-lp64d
linux-rv64gcv-lp64d-multilib rv32gcv-ilp32d, rv64gcv-lp64d

Target Information

Target Shorthand -march string
Bitmanip gc_zba_zbb_zbc_zbs

Notes

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

Copy link
Author

github-actions bot commented Dec 1, 2023

Lint Status

The following issues have been found with 27773-RISCV_Rename_and_unify_stringop_strategy_handling_NFC-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: dot, space, space, end of comment (4 error(s)) ===
gcc/config/riscv/riscv-opts.h:108:18:  /* No expansion.█*/
gcc/config/riscv/riscv-opts.h:110:38:  /* Use scalar expansion if possible.█*/
gcc/config/riscv/riscv-opts.h:112:39:  /* Only vector expansion if possible.█*/
gcc/config/riscv/riscv-opts.h:114:13:  /* Use any.█*/

=== ERROR type #2: lines should not exceed 80 characters (1 error(s)) ===
gcc/config/riscv/riscv.opt:555:80:Target RejectNegative Joined Enum(stringop_strategy) Var(stringop_strategy) Init(STRATEGY_AUTO)

=== ERROR type #3: there should be exactly one space between function name and parenthesis (6 error(s)) ===
gcc/config/riscv/riscv.opt:539:4:Name(stringop_strategy) Type(enum stringop_strategy_enum)
gcc/config/riscv/riscv.opt:543:4:Enum(stringop_strategy) String(auto) Value(STRATEGY_AUTO)
gcc/config/riscv/riscv.opt:546:4:Enum(stringop_strategy) String(libcall) Value(STRATEGY_LIBCALL)
gcc/config/riscv/riscv.opt:549:4:Enum(stringop_strategy) String(scalar) Value(STRATEGY_SCALAR)
gcc/config/riscv/riscv.opt:552:4:Enum(stringop_strategy) String(vector) Value(STRATEGY_VECTOR)
gcc/config/riscv/riscv.opt:555:33:Target RejectNegative Joined Enum(stringop_strategy) Var(stringop_strategy) Init(STRATEGY_AUTO)

Additional information

Copy link
Author

github-actions bot commented Dec 1, 2023

Apply Status

Target Status
Baseline hash: gcc-mirror/gcc@73e2bdb Applied
Tip of tree hash: gcc-mirror/gcc@12b67d1 Applied

Notes

Patch applied successfully

Additional information

Copy link
Author

github-actions bot commented Dec 1, 2023

Build GCC Status

Target Status
newlib-rv64gc-lp64d-multilib Success
linux-rv64gc_zba_zbb_zbc_zbs-lp64d-non-multilib Success
linux-rv64gcv-lp64d-multilib Success
linux-rv32gc_zba_zbb_zbc_zbs-ilp32d-non-multilib Success
newlib-rv64gcv-lp64d-multilib Success

Notes

Patch(es) were applied to the hash gcc-mirror/gcc@73e2bdb. If this patch commit depends on or conflicts with a recently committed patch, then these results may be outdated.

Additional information

Copy link
Author

github-actions bot commented Dec 1, 2023

Testsuite Status

Summary

New Failures gcc g++ gfortran Previous Hash
Resolved Failures gcc g++ gfortran Previous Hash
Unresolved Failures gcc g++ gfortran Previous Hash
linux: rv32 Bitmanip ilp32d medlow 99/84 41/26 13/3 gcc-mirror/gcc@73e2bdb
linux: rv32gcv ilp32d medlow multilib 110/98 47/29 18/6 gcc-mirror/gcc@73e2bdb
linux: rv64 Bitmanip lp64d medlow 29/14 15/6 12/2 gcc-mirror/gcc@73e2bdb
linux: rv64gcv lp64d medlow multilib 39/29 19/7 14/4 gcc-mirror/gcc@73e2bdb
newlib: rv32gc ilp32d medlow multilib 80/23 108/16 0/0 gcc-mirror/gcc@73e2bdb
newlib: rv64gc lp64d medlow multilib 69/18 81/12 0/0 gcc-mirror/gcc@73e2bdb
newlib: rv64gcv lp64d medlow multilib 74/30 85/13 0/0 gcc-mirror/gcc@73e2bdb

Additional information

@github-actions github-actions bot added the linter-failure Lint failed label Dec 1, 2023
@github-actions github-actions bot closed this as completed Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
linter-failure Lint failed
Projects
None yet
Development

No branches or pull requests

0 participants