<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Looks good to me<br>
<br>
On 17/01/14 18:03, Robert Lytton wrote:<br>
</div>
<blockquote
cite="mid:E55040AE4CA5DE4A84D2754CE295AF300168B183@EXMAILBOX1.vo.spidergroup.co.uk"
type="cite">
<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>
<div style="direction: ltr;font-family: Tahoma;color:
#000000;font-size: 10pt;">Hi,<br>
<br>
XCore target: addMemOperand as necessary<br>
<br>
BuildMI instructions were not including MachineMemOperand
information.<br>
This was discovered by
'MultiSource/Benchmarks/Prolangs-C/cdecl' failing<br>
due to a FrameIndex load incorrectly being hoisted by
postra-machine-licm.<br>
No other tests have been found to fail.<br>
<br>
I have attempted to write a test to highlight this bug
unsuccessfully.<br>
The original test-suite
'MultiSource/Benchmarks/Prolangs-C/cdecl' was the only test that
failed and now passes.<br>
<br>
As part of the trampoline, the reloading of R11 from a pre-stack
location in emitPrologue() is missing the addMemOperand().<br>
<br>
Robert<br>
<br>
</div>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Richard Osborne | XMOS
<a class="moz-txt-link-freetext" href="http://www.xmos.com">http://www.xmos.com</a>
</pre>
</body>
</html>