[Lldb-commits] [PATCH] D29347: Add ProcessLauncherNetBSD to spawn a tracee
Kamil Rytarowski via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Jan 31 13:47:38 PST 2017
krytarowski added a comment.
In https://reviews.llvm.org/D29347#662145, @labath wrote:
> This is also copying a lot of code only to make tiny changes to it. Please put the Linux process launcher into the posix folder (I suggest naming it ProcessLauncherPosixFork). Ifdef out the linux-specific parts (ASLR, personality) and you're done. :)
> For bonus points, you can put the ASLR code into a separate function, and then ifdef-out only that - should make main code flow better.
I will do it. Thank you for quick reply.
More information about the lldb-commits