XCore target: addMemOperand as necessary

Robert Lytton robert at xmos.com
Fri Jan 17 10:03:40 PST 2014


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140117/8572ce8c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PatchMemOperand
Type: application/octet-stream
Size: 15820 bytes
Desc: PatchMemOperand
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140117/8572ce8c/attachment.obj>


More information about the llvm-commits mailing list