diff options
| author | 2025-12-27 03:05:05 +0000 | |
|---|---|---|
| committer | 2026-02-16 19:11:24 +0000 | |
| commit | 8d4b47298b6c9eeb9a3f79a5b80d894f80ed4a45 (patch) | |
| tree | 63a8ea6f645b997fd7943a58a002e66244fa49ac /src/os.h | |
| parent | e7943fb834cbcaec585ed68e67b5f17ceda504f7 (diff) | |
| download | sst-8d4b47298b6c9eeb9a3f79a5b80d894f80ed4a45.tar.gz sst-8d4b47298b6c9eeb9a3f79a5b80d894f80ed4a45.zip | |
Do away with memcpy/memset/memcmp
Now everything is either explicitly a rep thing, or explicitly some
other small run of reasonably efficient instructions. No library calls
or function call overhead of any kind. Code size remains nice and small.
Ban a few other bad libc functions while we're at it. More can be added
later if we decide that's a useful idea.
This will require some discipline and workarounds to cope with Clang's
tendency to shove mem* calls in places you don't want them even when you
tell it not to. Well, c'est la vie.
I'm a little nervous that the __builtin_memmove() usage in shuntvars()
could still one day emit actual library calls. It appears we're not
importing memmove() from anywhere at the moment, though. So I guess it's
good enough for now.
Also I had to get rid of alloca() in con_.c but that was kind of bad
anyway, so that's fine.
Diffstat (limited to 'src/os.h')
| -rw-r--r-- | src/os.h | 5 |
1 files changed, 1 insertions, 4 deletions
@@ -120,10 +120,7 @@ typedef char os_char; #endif /* Copies n characters from src to dest, using the OS-specific char type. */ -static inline void os_spancopy(os_char *restrict dest, - const os_char *restrict src, int n) { - memcpy(dest, src, n * sizeof(os_char)); -} +void os_spancopy(os_char *restrict dest, const os_char *restrict src, int n); /* * Returns the last error code from an OS function - equivalent to calling |
