<div dir="ltr">Thank you.<br></div><div class="gmail_extra"><br><div class="gmail_quote">2014-09-25 16:45 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>
> 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>
</span><span class="">> Sent: Wednesday, September 24, 2014 7:08:48 PM<br>
> Subject: Re: br_cc definition<br>
><br>
><br>
</span><span class="">> I don't think so.<br>
<br>
</span>r218476. Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
 -Hal<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
><br>
><br>
> 2014-09-24 14:05 GMT-07:00 Hal Finkel < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> > :<br>
><br>
><br>
> ----- Original Message -----<br>
> > From: "deadal nix" < <a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a> ><br>
> > 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 4:03:20 PM<br>
> > Subject: Re: br_cc definition<br>
> ><br>
> ><br>
> > How do we get that in ?<br>
> ><br>
><br>
> Do you have commit access?<br>
><br>
> -Hal<br>
><br>
><br>
><br>
> ><br>
> > 2014-09-23 22:37 GMT-07:00 Hal Finkel < <a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a> > :<br>
> ><br>
> ><br>
> > ----- Original Message -----<br>
> > > From: "deadal nix" < <a href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a> ><br>
> > > 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.<br>
> > > 3<br>
> > > it the target of the branch. So SDTCisVT<0, OtherVT> and should<br>
> > > not<br>
> > > be an int.<br>
> > ><br>
> ><br>
> > Ah, you're right: Those are represented by CondCodeSDNode (not<br>
> > integer constants). LGTM.<br>
> ><br>
> > -Hal<br>
> ><br>
> ><br>
> ><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<br>
> > > > 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<br>
> > > 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>
> ><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>