<div dir="ltr">You're obviously much more familiar with the system than me, but at least from my point of view it kind of makes sense.  Target platform is just that, the target platform, which may not be the same as the selected platform.  I suppose one way to formalize it would be to add "selected platform" to the ExecutionContext.<div>
<br></div><div>Speaking of the ExecutionContext, I kind of dislike that it's stack-based and returned by value.  Many parts of LLDB aren't really const-correct, and returning by value necessitates passing it as to this method as a const-reference, and it then becomes difficult to access some of its functionality.  Not an issue in this particular CL, just an observation I had.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jul 7, 2014 at 2:59 PM,  <span dir="ltr"><<a href="mailto:jingham@apple.com" target="_blank">jingham@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
This seems okay.  It bugs me a little that I can get the platform two ways now, from the target & from the explicit platform.  It only makes sense if the two are the same, but it would be nice if there were some way to formalize that...<br>

<br>
Jim<br>
<div><div class="h5"><br>
> On Jul 7, 2014, at 1:21 PM, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
><br>
> Update the patch to take both an ExecutionContext and a Platform independently of each other.  The core logic has not changed, this particular validator still goes through the Platform and ignores the Target, since there may be no target running.<br>

><br>
> <a href="http://reviews.llvm.org/D4373" target="_blank">http://reviews.llvm.org/D4373</a><br>
><br>
> Files:<br>
>  include/lldb/Host/OptionParser.h<br>
>  include/lldb/Interpreter/CommandOptionValidators.h<br>
>  include/lldb/Interpreter/Options.h<br>
>  include/lldb/lldb-private-types.h<br>
>  source/Commands/CommandObjectPlatform.cpp<br>
>  source/Interpreter/Args.cpp<br>
>  source/Interpreter/CMakeLists.txt<br>
>  source/Interpreter/CommandOptionValidators.cpp<br>
>  source/Interpreter/Options.cpp<br>
</div></div>> <D4373.11127.patch>_______________________________________________<br>
<div class="HOEnZb"><div class="h5">> lldb-commits mailing list<br>
> <a href="mailto:lldb-commits@cs.uiuc.edu">lldb-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits</a><br>
<br>
</div></div></blockquote></div><br></div>