The required page alignment is different on different targets. On PowerPC, for example, we need 64K pages (the loader won't set different permissions on a finer granularity than that). I've set the existing targets to what I believe to be the correct values, and have updated the regression tests accordingly.
Diff Detail
Diff Detail
Event Timeline
ELF/Target.cpp | ||
---|---|---|
352 | Thanks for the investigation. Doing what gold does sounds reasonable. |
This is the same -- you want to set 0x1000 as default and assign only when a platform uses a different value. (I found I slightly prefer 4096 over 0x1000, but I can't explain why.)