<div dir="ltr"><div dir="ltr">Good catch, I've updated the comments in r351674, thanks!<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Jan 19, 2019 at 8:08 AM Eli Friedman <<a href="mailto:efriedma@codeaurora.org">efriedma@codeaurora.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/17/2019 11:31 PM, Dylan McKay via llvm-commits wrote:<br>
> Author: dylanmckay<br>
> Date: Thu Jan 17 23:31:34 2019<br>
> New Revision: 351526<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=351526&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=351526&view=rev</a><br>
> Log:<br>
> [AVR] Rewrite the CBRRdK instruction as an alias of ANDIRdK<br>
> +// imm_com8_XFORM - Return the complement of a t2_so_imm value<br>
> +def imm_com8_XFORM : SDNodeXForm<imm, [{<br>
> +  return CurDAG->getTargetConstant(~((uint8_t)N->getZExtValue()), SDLoc(N),<br>
> +                                   MVT::i8);<br>
> +}]>;<br>
<br>
The comment refers to t2_so_imm?<br>
<br>
> +<br>
> +// imm_com8 - Match an immediate that is a complement<br>
> +// of a 8-bit immediate.<br>
> +// Note: this pattern doesn't require an encoder method and such, as it's<br>
> +// only used on aliases (Pat<> and InstAlias<>). The actual encoding<br>
> +// is handled by the destination instructions, which use t2_so_imm.<br>
<br>
Also here, refers to t2_so_imm<br>
<br>
-Eli<br>
<br>
-- <br>
Employee of Qualcomm Innovation Center, Inc.<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project<br>
<br>
</blockquote></div>