PATCH. Fix ARM Leapcrel definition to make it loop hoistable

Yin Ma yinma at codeaurora.org
Mon Oct 7 11:39:26 PDT 2013


Hello,

 

This patch enables LEAPCREL to be moved out of a loop because it is
currently marked 

as hasSideEffects in .td file. Actually, it is not. By reading the comment
above this 

instruction. It should not be marked hasSideEffects from the very beginning.

 

Please review,

 

Yin 

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131007/2e50c9b5/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: leapcrel.diff
Type: application/octet-stream
Size: 1233 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131007/2e50c9b5/attachment.obj>


More information about the llvm-commits mailing list