managarm / mlibc

Portable C standard library
Other
846 stars 128 forks source link

Review architecture specific defines #1152

Open 64 opened 6 days ago

64 commented 6 days ago

Some architecture specific defines are really just checking for 32-bit / 64-bit: we should refactor these to directly check for pointer size so new ISA ports require less boilerplate.

e.g https://github.com/managarm/mlibc/blob/3b0bc20fd2e2d41545f1ddbe21f7652c56af8cd0/tests/ansi/strtol.c#L66-L76

VeerChaurasia commented 3 days ago

hey @64 can I take this one?

64 commented 3 days ago

Sure :)