<div dir="ltr">On 11 June 2013 21:19, Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span> wrote:<br><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"><div dir="auto"><div>I do not know either and that is my point :). I wanted to get more information on why value is tested and never used.<br>
</div><div>To me it is not relevant to use the value here. </div></div></blockquote><div><br></div><div style>I agree.</div><div><br></div><div><br></div><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">
<div dir="auto"><div><div class="im"><span style="color:rgb(34,34,34)">Agree, have you someone in mind that I could ping about that?</span></div></div></div></blockquote><div></div></div><br></div><div class="gmail_extra" style>
So, when in doubt, ask the code owner for the ARM back-end, which is Evan Cheng. Luckily, in this case, the original code is his anyway (r73413, 15 Jun 2009). ;)</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>
Evan,</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>Can you shed some light on what the getValue() is testing in this case?</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>
<div class="gmail_extra">+  // Must sure the base address satisfies i64 ld / st alignment requirement.</div><div class="gmail_extra">+  if (!Op0->hasOneMemOperand() ||</div><div class="gmail_extra">+      !Op0->memoperands_begin()->getValue() ||</div>
<div class="gmail_extra">+      Op0->memoperands_begin()->isVolatile())</div><div><br></div></div><div class="gmail_extra" style>cheers,</div><div class="gmail_extra" style>--renato</div></div>