<html dir="ltr">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>
</head>
<body ocsi="0" fpstyle="1">
<div style="direction: ltr;font-family: Tahoma;color: #000000;font-size: 10pt;">thank you!<br>
Added to CC.<br>
Not sure who I should request as a reviewer.<br>
<br>
Robert<br>
<br>
<div style="font-family: Times New Roman; color: #000000; font-size: 16px">
<hr tabindex="-1">
<div style="direction: ltr;" id="divRpF616200"><font color="#000000" face="Tahoma" size="2"><b>From:</b> David Blaikie [dblaikie@gmail.com]<br>
<b>Sent:</b> 24 January 2014 16:33<br>
<b>To:</b> Robert Lytton<br>
<b>Cc:</b> echristo@gmail.com; llvmdev@cs.uiuc.edu<br>
<b>Subject:</b> Re: [LLVMdev] test suite 'owner'<br>
</font><br>
</div>
<div></div>
<div>
<div dir="ltr">If you haven't already, you probably want to add llvm-commits as a reviewer or cc on these code reviews so they can be reviewed in public.</div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On Fri, Jan 24, 2014 at 6:37 AM, Robert Lytton <span dir="ltr">
<<a href="mailto:robert@xmos.com" target="_blank">robert@xmos.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
<div>
<div style="direction:ltr; font-size:10pt; font-family:Tahoma">I have unbundled and submitted the first 4 patches.<br>
Please see:<br>
<br>
    <font><span style="font-size:10pt"><a href="http://llvm-reviews.chandlerc.com/D2615" target="_blank">http://llvm-reviews.chandlerc.com/D2615</a></span></font><br>
    [PATCH] test-stuite: Fix signature of main() to use 'int' rather than 'int32_t'<br>
<br>
    <font><span style="font-size:10pt"><a href="http://llvm-reviews.chandlerc.com/D2617" target="_blank">http://llvm-reviews.chandlerc.com/D2617</a><br>
    </span></font>[PATCH] test-suite: Fix name clash 'infinity'<br>
<br>
    <font><span style="font-size:10pt"><a href="http://llvm-reviews.chandlerc.com/D2618" target="_blank">http://llvm-reviews.chandlerc.com/D2618</a><br>
    </span></font><font><span style="font-size:10pt">[PATCH] test-suite: Fix name clash 'log2'<br>
<br>
    </span></font><font><span style="font-size:10pt"><a href="http://llvm-reviews.chandlerc.com/D2619" target="_blank">http://llvm-reviews.chandlerc.com/D2619</a><br>
</span></font><font><span style="font-size:10pt">    [PATCH] test-suite: XCore target: fix handling of <string.h> and <memory.h><br>
<br>
Thank you.<br>
</span></font>
<div class="im">
<div style="font-size:16px; font-family:Times New Roman">
<div>
<div style="direction:ltr; font-size:10pt; font-family:Tahoma">
<div style="font-size:16px; font-family:Times New Roman">
<div>Robert<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex; border-left:1px #ccc solid; padding-left:1ex">
<div>
<div style="direction:ltr; font-size:10pt; font-family:Tahoma">
<div style="font-size:16px; font-family:Times New Roman">
<hr>
<div style="direction:ltr"><font color="#000000" face="Tahoma"><b>From:</b> Eric Christopher [<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>]<br>
<b>Sent:</b> 13 January 2014 19:16<br>
<b>To:</b> Robert Lytton; <a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a><br>
<b>Subject:</b> [LLVMdev] test suite 'owner'<br>
</font><br>
</div>
</div>
</div>
</div>
<div>
<div style="direction:ltr; font-size:10pt; font-family:Tahoma">
<div style="font-size:16px; font-family:Times New Roman">
<div></div>
<div>Some of these are pretty weird, e.g. int32_t main. Probably the best thing is to submit each patch individually with an explanation of what the purpose is and we can talk about them then.
<div><br>
</div>
<div>-eric</div>
<div>
<div><br>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
</div>
</body>
</html>