<div dir="ltr"><div style><div>On 21 February 2013 09:20, Keith Walker <span dir="ltr"><<a href="mailto:keith.walker@arm.com" target="_blank">keith.walker@arm.com</a>></span> wrote:<br></div></div><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">This patch fixes the encoding of the constant value in ARM "adr rN, #Const"<br>

instruction which currently asserts on some "Const" values that can be<br>
validly encoded in the instruction.<br></blockquote><div><br></div>Hi Keith,<div><br></div><div>Patch looks good. I'd just initialize the variable to zero so that future code movements only introduce bugs, not segfaults. ;)</div>
<div><br></div><div><div>+  int SoImmVal = 0;</div><div>   if (offset == INT32_MIN) {</div><div>     Val = 0x1000;</div><div>-    offset = 0;</div></div><div><br></div><div style>Also, maybe add one-line comments on what types of values will match in each case.</div>
<div><br></div><div style>cheers,</div><div style>--renato </div></div></div></div>