[PATCH] D24987: [ELF] Use MaxPageSize for aligning PT_LOAD

Petr Hosek via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 14:35:03 PDT 2016

Previous message: [PATCH] D24987: [ELF] Use MaxPageSize for aligning PT_LOAD
Next message: [PATCH] D24987: [ELF] Use MaxPageSize for aligning PT_LOAD
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

phosek added a comment.

The question is whether it wouldn't also make more sense to rename `MaxPageSize` and `PageSize` to something more descriptive/less confusing, e.g. gold uses `abi_pagesize` (for `MaxPageSize`) and `common_pagesize` (for `PageSize`) which IMHO makes more sense.


https://reviews.llvm.org/D24987

Previous message: [PATCH] D24987: [ELF] Use MaxPageSize for aligning PT_LOAD
Next message: [PATCH] D24987: [ELF] Use MaxPageSize for aligning PT_LOAD
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list