[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.


Repository:
  rL LLVM

https://reviews.llvm.org/D29347





More information about the lldb-commits mailing list