<div dir="ltr">How do we get that in ?<br></div><div class="gmail_extra"><br><div class="gmail_quote">2014-09-23 22:37 GMT-07:00 Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">----- Original Message -----<br>
> From: "deadal nix" <<a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a>><br>
</span><span class="">> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>><br>
> Cc: "llvm-commits" <<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a>><br>
> Sent: Wednesday, September 24, 2014 12:29:12 AM<br>
> Subject: Re: br_cc definition<br>
><br>
><br>
> The argument 0 is the condition code type. 1 and 2 are arguments. 3<br>
> it the target of the branch. So SDTCisVT<0, OtherVT> and should not<br>
> be an int.<br>
><br>
<br>
</span>Ah, you're right: Those are represented by CondCodeSDNode (not integer constants). LGTM.<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
><br>
> 2014-09-23 5:42 GMT-07:00 Hal Finkel < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> > :<br>
><br>
><br>
><br>
><br>
> ----- Original Message -----<br>
> > From: "deadal nix" < <a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a> ><br>
> > To: "llvm-commits" < <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a> ><br>
> > Sent: Tuesday, September 23, 2014 3:20:59 AM<br>
> > Subject: br_cc definition<br>
> ><br>
> ><br>
> ><br>
> > As per<br>
> > <a href="http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-August/075321.html" target="_blank">http://lists.cs.uiuc.edu/pipermail/llvmdev/2014-August/075321.html</a><br>
> > ,<br>
> > it look like I'm not the only one to want to match br_cc directly.<br>
> > So I prepared a patch.<br>
><br>
> +def SDTBrCC : SDTypeProfile<0, 4, [ // brcc<br>
> + SDTCisVT<0, OtherVT>, SDTCisSameAs<1, 2>, SDTCisVT<3, OtherVT><br>
> +]>;<br>
> +<br>
><br>
> Shouldn't the type constraint for the condition code be SDTCisInt<0>?<br>
><br>
> -Hal<br>
><br>
> ><br>
> > _______________________________________________<br>
> > llvm-commits mailing list<br>
> > <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
> ><br>
><br>
> --<br>
> Hal Finkel<br>
> Assistant Computational Scientist<br>
> Leadership Computing Facility<br>
> Argonne National Laboratory<br>
><br>
><br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</div></div></blockquote></div><br></div>