[PATCH] [BUG 22755] Fix bugs in CombinToPreIndexedLoadStore

Yin Ma yinma at codeaurora.org
Wed Mar 11 10:40:45 PDT 2015


Hi,

 

This simple change is for Bug 22755

https://llvm.org/bugs/show_bug.cgi?id=22755

 

Basically, the code tried to fold two adds. However, in a situation where it
is a preindex load L converted before, it is possible that the input of one
add is from the result of L and the input of another add is from the updated
index of L. These two adds cannot be combined together clearly, so we need
check to make sure two adds are from the same ResNo by adding code in the
patch.

 

Unfortunately, I don't have a test case for it. However, it should be quite
clear to see the change.

Please review.

 

Thanks,

 

Yin 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150311/4ec65737/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ResNo.diff
Type: application/octet-stream
Size: 530 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150311/4ec65737/attachment.obj>


More information about the llvm-commits mailing list