<p>LGTM</p>
<div class="gmail_quote">On Apr 2, 2012 10:00 AM, "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Mon, 2 Apr 2012 09:42:45 -0700<br>
Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
<br>
> There are unit teats for the triple class.<br>
<br>
Indeed, thanks! Updated patch attached.<br>
<br>
 -Hal<br>
<br>
> On Apr 2, 2012 9:41 AM, "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
><br>
> > On Mon, 2 Apr 2012 09:19:47 -0700<br>
> > Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
> ><br>
> > > Can you add some tests?<br>
> ><br>
> > Sure, where would they go?<br>
> ><br>
> >  -Hal<br>
> ><br>
> > > On Apr 2, 2012 8:45 AM, "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br>
> > ><br>
> > > > The attached patch adds support for the triples used on the IBM<br>
> > > > BG/P and BG/Q supercomputers. Please let me know if these are<br>
> > > > okay to commit.<br>
> > > ><br>
> > > > Thanks again,<br>
> > > > Hal<br>
> > > ><br>
> > > > --<br>
> > > > Hal Finkel<br>
> > > > Postdoctoral Appointee<br>
> > > > Leadership Computing Facility<br>
> > > > Argonne National Laboratory<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>
> ><br>
> ><br>
> > --<br>
> > Hal Finkel<br>
> > Postdoctoral Appointee<br>
> > Leadership Computing Facility<br>
> > Argonne National Laboratory<br>
> ><br>
<br>
<br>
<br>
--<br>
Hal Finkel<br>
Postdoctoral Appointee<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</blockquote></div>