<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 9, 2015 at 4:09 PM, Pete Cooper <span dir="ltr"><<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Hi all<div><br></div><div>Saleem, in 2014 you added the .thumb_set test case.  I’ve found a difference in behaviour of .thumb_set compared to .set.  I was hoping to resolve this difference as it will allow me to reapply r239440.</div><div><br></div><div>The issue i’m seeing is that we define an alpha function, then beta function, then assign alpha to beta.  But given that beta has already been defined, this would mean we are redefining it at the point of the .thumb_set:</div><div><br></div><div><div><span style="white-space:pre-wrap">        </span>.text</div><div><span style="white-space:pre-wrap">    </span>.thumb</div><div><br></div><div><span style="white-space:pre-wrap">  </span>.thumb_set alias_defined_data, bedazzle</div><div><br></div><div><span style="white-space:pre-wrap"> </span>.type alpha,%function</div><div>alpha:</div><div><span style="white-space:pre-wrap">       </span>nop</div><div><br></div><div>        .type beta,%function</div><div>beta:</div><div><span style="white-space:pre-wrap">  </span>bkpt</div><div><br></div><div><span style="white-space:pre-wrap">    </span>.thumb_set beta, alpha</div></div><div><br></div><div>The above code currently passes.  However, if I change .thumb_set to .set then I get</div><div><br></div><div><div style="margin:0px;font-size:11px;font-family:Menlo"><b>../test/MC/ARM/thumb_set.s:60:13: </b><span style="color:#c33720"><b>error: </b></span><b>redefinition of 'beta'</b></div><div style="margin:0px;font-size:11px;font-family:Menlo">        .set beta, alpha</div></div><div style="margin:0px;font-size:11px;font-family:Menlo"><br></div><div style="margin:0px;font-size:11px;font-family:Menlo">I would like to make .thumb_set throw the same error here as I believe that is how it should behave in this case.</div><div style="margin:0px;font-size:11px;font-family:Menlo"><br></div><div style="margin:0px;font-size:11px;font-family:Menlo">Please let me know if you are ok with this, or some other approach.  I’m unfamiliar with it so its possible that what i’ve found isn’t a bug at all but just how we want it to behave.</div></div></blockquote><div><br></div><div>The behavior was taken from GAS.  I don't know if there are applications using this and dependent on the behavior.  Given that this is a GNU extension, I think its better to keep the behavior similar to GAS rather than implement and diverge.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div style="margin:0px;font-size:11px;font-family:Menlo">Cheers,</div><div style="margin:0px;font-size:11px;font-family:Menlo">Pete</div></div></blockquote></div><br>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>