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

x86: 64-bit time #994

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

x86: 64-bit time #994

wants to merge 6 commits into from

Conversation

FedorLap2006
Copy link
Contributor

Currently time_t is long, which is fine 64 bit arches, since long is always 8 bytes wide.
However, on x86 long is 4 bytes wide.

This PR changes time_t to always be 8 bytes wide, by making it __mlibc_int64.

@FedorLap2006
Copy link
Contributor Author

FedorLap2006 commented Feb 17, 2024

After a bit of research, I've found out this problem is not so trivial, since linux ABI on x86 requires time_t to be long
(4 bytes wide)

@FedorLap2006 FedorLap2006 changed the title Make time_t always use 64 bit signed integer x86: 64-bit time Mar 4, 2024
sysdeps/linux/generic/sysdeps.cpp Outdated Show resolved Hide resolved
options/internal/include/bits/field-padding.h Outdated Show resolved Hide resolved
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.

2 participants