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