<div dir="ltr"><div><div><div>Hi Quentin,<br><br></div>Yes, you are correct. Now I split it into two separate patches.<br><br></div>One is at <a href="http://reviews.llvm.org/D3431">http://reviews.llvm.org/D3431</a> for enabling global merge pass, the other is at <a href="http://reviews.llvm.org/D3432">http://reviews.llvm.org/D3432</a> for implementing ADRP CSE for global symbols.<br>
<br></div>I don't use the original code review at <a href="http://reviews.llvm.org/D3223" target="_blank">http://reviews.llvm.org/D3223</a> to avoid confusion.<br><div><br></div><div>Thanks,<br></div><div>-Jiangning<br>
</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-04-19 0:48 GMT+08:00 Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
  Hi Jianging,<br>
<br>
  It looks to me that you have two change sets in the same patch:<br>
  - One that adds the support of external linkage to the global merge pass.<br>
  - One that enables the global merge pass for AArch64.<br>
<br>
  Could you split the patch to match that?<br>
<br>
  Thanks,<br>
  -Quentin<br>
<br>
<a href="http://reviews.llvm.org/D3223" target="_blank">http://reviews.llvm.org/D3223</a><br>
<br>
<br>
</blockquote></div><br></div>