[PATCH] Implement ADRP CSE for global symbols
Jiangning Liu
liujiangning1 at gmail.com
Mon Apr 21 22:50:24 PDT 2014
This patch is to fix the issue raised by Quentin and Tim. The code was cleaned, and the followings are changed
1) Change IsInternalOnly to IsExternal, and add assertion.
1) The behavior of -global-merge-on-external will not affect static variables any longer.
2) Split a long if condition expression.
Thanks,
-Jiangning
Hi qcolombet, t.p.northover,
http://reviews.llvm.org/D3432
CHANGE SINCE LAST DIFF
http://reviews.llvm.org/D3432?vs=8660&id=8710#toc
Files:
include/llvm/IR/GlobalAlias.h
lib/CodeGen/AsmPrinter/AsmPrinter.cpp
lib/IR/Globals.cpp
lib/Transforms/Scalar/GlobalMerge.cpp
test/CodeGen/AArch64/global_merge_1.ll
test/CodeGen/AArch64/global_merge_2.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3432.2.patch
Type: text/x-patch
Size: 9042 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140421/5fd19328/attachment.bin>
More information about the llvm-commits
mailing list