<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Jun 20, 2013 at 10:58 AM, Tom Stellard <span dir="ltr"><<a href="mailto:tom@stellard.net" target="_blank">tom@stellard.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Sean,<br>
<br>
Attached is an updated patch which addresses your comments.<br>
<div class="im"><br>
On Wed, Jun 19, 2013 at 12:52:03PM -0700, Sean Silva wrote:<br>
> +.. code-block:: llvm<br>
><br>
> This should be a `c++` code block, since this is c++ code (this determines<br>
> how it is syntax-highlighted).<br>
><br>
> +  int DIndex = SP::getNamedOperandIdx(SP::XNORrr, SP::OpName::d);     //<br>
> => -1<br>
><br>
> Does it even make sense to return -1 for "not found"?<br>
><br>
<br>
</div>Do you think we should be returning some other value for "not found"?<br></blockquote><div><br></div><div style>Sorry, let me rephrase. Does -1 ever get returned, or is it a "can't happen"? My intuition is that generally if you are calling this function you need a valid result, right?</div>
<div style><br></div><div style>-- Sean Silva </div></div></div></div>