[PATCH] D19761: [safestack] Support runtime_init attribute in SafeStack pass
Evgeniy Stepanov via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 20 12:41:59 PDT 2016
eugenis added a comment.
Do you really need all this complexity, and even a magic function attribute (which is btw entirely safestack-specific, so should have "safestack" in the name)?
We've had the same problem on Android, and were able to solve it entirely on the libc side:
Alternatively, consider moving this code into a runtime library function. Then SafeStack instrumentation would just call a function in the prologue instead of reading from a global. This would be good for code size.
More information about the llvm-commits