<div dir="ltr">Hi Chad,<div><br></div><div>Your comment says "// TBNZ only operates on i64's, but the ext should be free.", so why TBZ/TBNZ only operates on 64-bit?</div><div><br></div><div>ARMARM says both W and X are supported.</div>
<div><br></div><div><div>+        if (LHS.getValueType() == MVT::i32)</div><div>+          LHS = DAG.getAnyExtOrTrunc(LHS, dl, MVT::i64);</div></div><div><br></div><div>This promotion finally will be transformed back to 32-bit version by this pat, I think.</div>
<div><br></div><div><div>  def : Pat<(node GPR64:$Rn, tbz_imm0_31_diag:$imm, bb:$target),</div><div>            (!cast<Instruction>(NAME#"W") (EXTRACT_SUBREG GPR64:$Rn, sub_32),</div><div>            tbz_imm0_31_diag:$imm, bb:$target)>;</div>
</div><div><br></div><div>Your code logic doesn't have issue, but at least the comment is misleading, so could you please remove it?</div><div><br></div><div>And if you like, could you please change to use 32-bit version of TBZ/TBNZ for 32-bit case directly?</div>
<div><br></div><div>Thanks,</div><div>-Jiangning</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-07-30 11:21 GMT+08:00 Jiangning Liu <span dir="ltr"><<a href="mailto:liujiangning1@gmail.com" target="_blank">liujiangning1@gmail.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">Hi Chad,<br>
<br>
I'm happy with this, and your new patch looks good to me if only you can<br>
add more comments in the code around excluding AND, because at the first<br>
look it's strange.<br>
<br>
Thanks,<br>
-Jiangning<br>
<br>
2014-07-30 0:02 GMT+08:00 Chad Rosier <<a href="mailto:mcrosier@codeaurora.org">mcrosier@codeaurora.org</a>>:<br>
<br>
> >>! In D4440#12, @mcrosier wrote:<br>
> > The performance numbers are slightly better, but still less than noise.<br>
>  All, please take a look.<br>
><br>
> To be clear, the numbers are slightly better relative to the ADD/SUB only<br>
> combine.  Overall, we still see a large improvement in eembc/OAv2.<br>
><br>
> <a href="http://reviews.llvm.org/D4440" target="_blank">http://reviews.llvm.org/D4440</a><br>
><br>
><br>
><br>
<br>
</div></div><a href="http://reviews.llvm.org/D4440" target="_blank">http://reviews.llvm.org/D4440</a><br>
<br>
<br>
</blockquote></div><br></div>