[Lldb-commits] [PATCH] D46934: Make ObjectFileMachO work on non-darwin platforms

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed May 16 09:02:24 PDT 2018


labath added inline comments.


================
Comment at: source/Plugins/Process/Utility/RegisterContextDarwin_arm.cpp:1037
     SetError(set, Write, -1);
-    return KERN_INVALID_ARGUMENT;
+    return -1;
   }
----------------
aprantl wrote:
> Could we keep this as a local constant?
> perhaps with an #ifndef KERN_INVALID_ARGUMENT clause?
We could. I didn't do that originally, as `RegisterContextDarwin_x86_64::WriteGPR()` looks exactly like this function, and it uses `-1` already, but that works for me as well.


https://reviews.llvm.org/D46934





More information about the lldb-commits mailing list