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

[libc] Fix PATH bug in execvpe, update ewcc includes, speed up amalloc #2173

Merged
merged 2 commits into from
Jan 4, 2025

Conversation

ghaerr
Copy link
Owner

@ghaerr ghaerr commented Jan 4, 2025

When adding "time" to each of the build steps in ghaerr/8086-toolchain#31, it was found that the PATH wasn't being set properly for the execvpe system call.

Slightly increase speed of amalloc in C86 and AS86 by not searching from the start of the arena each time. This delays free block merging until later, but shouldn't be a problem for amalloc, since it is a fixed size heap anyways. The search was originally set to start from the arena start so that the heap wouldn't continue to grow endlessly before having to sbrk to expand the heap, which is still being done in v7 malloc (debug malloc). Basically we have a classic time/space tradeoff, and amalloc is now tuned for time, whereas debug malloc is tuned for space. Since AS86 does 700+ mallocs each pass, this is worth it and the amalloc arena size is fixed (for now) anyways.

Removes searching for WATCOM headers in ewcc. Discussed in #2162 (reply in thread).

Adds libc/include/watcom/float.h for building basic.c.

@ghaerr ghaerr merged commit 794ed00 into master Jan 4, 2025
2 checks passed
@ghaerr ghaerr deleted the execvpe branch January 4, 2025 02:40
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.

1 participant