[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