<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 15 January 2014 12:54, Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>></span> wrote:<br>
<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="ltr"><div style="font-family:arial,helvetica,sans-serif;font-size:small">
For llvm.arm.neon.vld3.v1i64 plus 3*64 offset writeback, we can use vld1 with writeback to implement the same functionality, because ARM instruction vld1 accept 64-bit vector list.<br></div></div></blockquote><div><br></div>
<div>I understand that this is a different issue, but do we have to do this? Can't we just emit vld/vst 2/3/4?</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="ltr"><div style="font-family:arial,helvetica,sans-serif;font-size:small">// We use a VST1 for v1i64 even if the pseudo says vld2/3/4, so<br>// check for that explicitly too. Horribly hacky, but temporary.<br></div>
</div></blockquote><div><br></div><div><div>I'd at least like to have a "FIXME" on that "temporary" comment, so that we can find it easier, later.</div></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="ltr"><div style="font-family:arial,helvetica,sans-serif;font-size:small">I introduced two new function isVLDfixed and isVSTfixed to cover all _fixed ISDNode.</div></div></blockquote><div><br></div><div>On those functions, a switch would be a bit less verbose and clearer.</div>
<div><br></div><div>As a quick fix, the patch is ok (with the comments above), but would be good to at least have an idea of how much work is involved in emitting vld/vst 2/3/4.</div></div><br></div><div class="gmail_extra">
cheers,</div><div class="gmail_extra">--renato</div></div>