Skip to content

binutils: Track RISC-V pr19719 test failures - #1562

Closed
jay23606 wants to merge 1 commit into
openRuyi-Project:mainfrom
jay23606:pr-go/1558-binutils-risc-v-ld-test-run-pr19719-fun-
Closed

jay23606 wants to merge 1 commit into
openRuyi-Project:mainfrom
jay23606:pr-go/1558-binutils-risc-v-ld-test-run-pr19719-fun-

Conversation

@jay23606

Copy link
Copy Markdown

Bug

The binutils %check workaround deleted both pr19719 tests unconditionally, masking a known RISC-V failure during the PGO training run.

Change

Keep both tests enabled and mark them as expected failures on RISC-V while upstream ld/32983 remains unresolved.

Testing

  • python tests/test_binutils_pr19719_workaround.py (passed)
  • git diff --check (passed)

untested: CONTRIBUTING does not mention tests

Fixes #1558

@openruyi-bot openruyi-bot Bot added Community: Contribution Issues that are actively being worked on by our community. BuildSystem: autotools Issue/PR related to autotools BuildSystem. Target: Rolling Targeting rolling/current branch. Workflow: Ready For Review Ready for review by anyone with domain expertise. labels Oct 31, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The comment "?" does not specify a question or requested change I can address.

@openruyi-bot openruyi-bot Bot added Workflow: In Review Issues that are undergoing code review and/or undergoing design review. and removed Workflow: Ready For Review Ready for review by anyone with domain expertise. labels Oct 31, 2026
@misaka00251 misaka00251 closed this Nov 2, 2026
@openruyi-bot openruyi-bot Bot removed the Workflow: In Review Issues that are undergoing code review and/or undergoing design review. label Nov 2, 2026
@misaka00251

misaka00251 commented Nov 2, 2026 •

Copy link
Copy Markdown
Member

This Pull Request is extremely poor quality. It appears to be generated by an LLM (or agent), as it does not adhere to the guidelines outlined in agents.md, fails to meet our packaging standards, and contains a mistake that is obvious upon first glance. Furthermore, the author of the Pull Request (or the LLM) seems completely unaware of these issues, even after the maintainer pointed them out.

I will be closing this Pull Request. We seek quality contributions, not subpar submissions generated by AI that haven't been reviewed by a human.

@openruyi-bot openruyi-bot Bot added the Workflow: In Review Issues that are undergoing code review and/or undergoing design review. label Nov 2, 2026
@jay23606
jay23606 deleted the pr-go/1558-binutils-risc-v-ld-test-run-pr19719-fun- branch November 2, 2026 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BuildSystem: autotools Issue/PR related to autotools BuildSystem. Community: Contribution Issues that are actively being worked on by our community. Target: Rolling Targeting rolling/current branch. Workflow: In Review Issues that are undergoing code review and/or undergoing design review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

binutils: RISC-V ld test "Run pr19719 fun defined" fails during the PGO training run and is hidden by the %check workaround

3 participants