[PATCH] D24891: [ELF] Support -z max-page-size option

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 27 18:56:14 PDT 2016


rafael added inline comments.

================
Comment at: ELF/Driver.cpp:671
@@ +670,3 @@
+  if (Config->MaxPageSize != Target->MaxPageSize)
+    Target->PageSize = Config->MaxPageSize;
+
----------------
Why do you need to set bits in Target? It would be more natural to change users to use the Config value and default that to the Target one.


https://reviews.llvm.org/D24891





More information about the llvm-commits mailing list