Skip to content

echo, false: replace benchmark#12091

Open
oech3 wants to merge 1 commit intouutils:mainfrom
oech3:echo-bench
Open

echo, false: replace benchmark#12091
oech3 wants to merge 1 commit intouutils:mainfrom
oech3:echo-bench

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented Apr 30, 2026

Preliminary for removing Box dyn.

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Congrats! The gnu test tests/cut/bounded-memory is now passing!
Congrats! The gnu test tests/printf/printf-surprise is now passing!

@oech3 oech3 marked this pull request as ready for review April 30, 2026 04:49
uu_true,
uu_false
uu_true
# uu_false is mostly same with true
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why removing it ?

Copy link
Copy Markdown
Contributor Author

@oech3 oech3 Apr 30, 2026

Choose a reason for hiding this comment

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

Because false is mostly same score with true.
I wanted to save binaries CI builds.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

we do have issues with rate limiting for these jobs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants