[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 mailing list