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><br><div>On Fri Jan 10 2014 at 4:13:47 AM, Robert Lytton <<a href="mailto:robert@xmos.com">robert@xmos.com</a>> wrote:</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div>
<div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt">Hi,<br>
<br>
I have found it necessary to make some changes to the test-suite for the XCore platform.<br>
<br>
These changes include:<br>
altering #includes, as supported by XCore;<br>
using stdout or stderr to make the output diffs consistent (fixing expected output too);<br>
(This work is still under review as best way to do it)<br>
'fixing' symbol and type problems e.g name clashes & scope;<br>
altering/adding code snippets and macros.<br>
<br>
I have used #ifdef to limit and keep any changes specific to the XCore.<br>
Some of these could/should be made common to all targets e.g. log2() -> logTwo().<br>
<br>
I have also altered the Makefile to filter out tests not supported by the XCore.<br>
<br>
I would like to discuss the changes I have found necessary to make and what is the next step.<br>
Should any/all of them be pushed upstream?</div></div><div><div style="direction:ltr;font-family:Tahoma;color:#000000;font-size:10pt"><br>
<br>
Robert<br>
<br>
</div></div></blockquote>