Fix bug:17136: RescheduleOps of ARMLoadStoreOptimizer

[Weiming Zhao]

Hi,

 

This patches fixes bug 17136 (ARMLoadStoreOptimizer):

 

In ARMPreAllocLoadStoreOpt::RescheduleOps():

 

The loop below is supposed to insert top n Ops into MemOps. But now it's
inserting bottom n Ops:

 

    for (int i = NumMove-1; i >= 0; --i) {

        MemOps.insert(Ops[i]);

        MemRegs.insert(Ops[i]->getOperand(0).getReg());

      }

 

It should be

for (int i = NumMove-1; i >= 0; --i) {

   MemOps.insert(Ops[Ops.size() - 1 - i]);

   MemRegs.insert(Ops[Ops.size() - 1 - i]->getOperand(0).getReg());

  }

 

Please help to review the patch.

 

Thanks,

Weiming

 

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130924/ae1f6ceb/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-PR-17136-Fix-RescheduleOps-of-ARMLoadStoreOptimizer.patch
Type: application/octet-stream
Size: 1089 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130924/ae1f6ceb/attachment.obj>