<div dir="ltr">Hi Tim,<div><br></div><div>The reason changing <span style="font-family:arial,sans-serif;font-size:13.63636302947998px">fpz32short to </span><span style="font-family:arial,sans-serif;font-size:13.63636302947998px">fpz32 is that we think it's a bug in ARMARM: all floating point zero should use unique syntax, and </span><span style="font-family:arial,sans-serif;font-size:13.63636302947998px">obviously</span><span style="font-family:arial,sans-serif;font-size:13.63636302947998px"> #0.0 is more appropriate than #0.  ARMARM will fix this in future.</span></div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/12/6 Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
  Hi Kevin,<br>
<br>
  I'm glad you're in the area! I was looking at these just yesterday and noticed that the syntax they use for printing is non-canonical. The zero should apparently print as "#0" rather than "#0.0"<br>

<br>
  Looks like there was some miscommunication in the code-review when they were committed. You originally suggested an "fpz32short" operand class for them, which printed as "#0" but was otherwise identical to fpz32. That was a good idea, and as I recall the only problem was that it had an unneeded "PrintMethod", but it somehow got dropped from the final patch.<br>

<br>
  Cheers.<br>
<br>
  Tim.<br>
<br>
  (I'm also horrified by that Neon_cmpz: it really shouldn't exist  as a separate AArch64ISD node from Neon_cmp. But we can probably leave that until the two backends are merged).<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D2349" target="_blank">http://llvm-reviews.chandlerc.com/D2349</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Best Regards,<div><br></div><div>Kevin Qin</div></div>
</div>