[PATCH] 'CSE' of ADRP instructions used for load/store addressing

[Jiangning Liu]

Hi Quentin,

Yes, you are correct. Now I split it into two separate patches.

One is at http://reviews.llvm.org/D3431 for enabling global merge pass, the
other is at http://reviews.llvm.org/D3432 for implementing ADRP CSE for
global symbols.

I don't use the original code review at http://reviews.llvm.org/D3223 to
avoid confusion.

Thanks,
-Jiangning



2014-04-19 0:48 GMT+08:00 Quentin Colombet <qcolombet at apple.com>:

>
>   Hi Jianging,
>
>   It looks to me that you have two change sets in the same patch:
>   - One that adds the support of external linkage to the global merge pass.
>   - One that enables the global merge pass for AArch64.
>
>   Could you split the patch to match that?
>
>   Thanks,
>   -Quentin
>
> http://reviews.llvm.org/D3223
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140419/740c6197/attachment.html>