<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">LGTM. <div><br><div><div>On Apr 2, 2013, at 8:28 AM, Arnold Schwaighofer <<a href="mailto:aschwaighofer@apple.com">aschwaighofer@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">DAGCombiner: Merge store/loads when we have extload/truncstores<br><br>This is helps on architectures where i8,i16 are not legal but we have byte,<br>and short loads/stores like on ARM.<br><br>Allowing us to merge copies like the one below on ARM.<br><br>copy(char *a, char *b, int n) {<br> do {<br> int t0 = a[0];<br> int t1 = a[1];<br> b[0] = t0;<br> b[1] = t1;<br><br><a href="radar://13536387">radar://13536387</a><br><br><span><0001-DAGCombiner-Merge-store-loads-when-we-have-extload-t.patch></span></div></blockquote></div><br></div></body></html>