XCore target: addMemOperand as necessary

[Richard Osborne]

Looks good to me

On 17/01/14 18:03, Robert Lytton wrote:
> Hi,
>
>     XCore target: addMemOperand as necessary
>
>     BuildMI instructions were not including MachineMemOperand information.
>     This was discovered by 'MultiSource/Benchmarks/Prolangs-C/cdecl' 
> failing
>     due to a FrameIndex load incorrectly being hoisted by 
> postra-machine-licm.
>     No other tests have been found to fail.
>
> I have attempted to write a test to highlight this bug unsuccessfully.
> The original test-suite 'MultiSource/Benchmarks/Prolangs-C/cdecl' was 
> the only test that failed and now passes.
>
> As part of the trampoline, the reloading of R11 from a pre-stack 
> location in emitPrologue() is missing the addMemOperand().
>
> Robert
>


-- 
Richard Osborne | XMOS
http://www.xmos.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140217/77d1942b/attachment.html>