[PATCH] Implement ADRP CSE for global symbols

Jiangning Liu liujiangning1 at gmail.com
Tue May 6 18:49:51 PDT 2014


Hi Quentin,

Sorry for my late response! I was in vacation and busy at other tasks.

I think I totally agree with you, so I added a target hook returning the alignment required by global merge on external, and in global merge pass there is an assertion to make sure the alignment returned by target hook always meets natural alignment requirement.

Thanks,
-Jiangning

http://reviews.llvm.org/D3432

Files:
  include/llvm/IR/GlobalAlias.h
  include/llvm/Target/TargetLowering.h
  lib/CodeGen/AsmPrinter/AsmPrinter.cpp
  lib/IR/Globals.cpp
  lib/Target/AArch64/AArch64ISelLowering.cpp
  lib/Target/AArch64/AArch64ISelLowering.h
  lib/Target/ARM64/ARM64ISelLowering.cpp
  lib/Target/ARM64/ARM64ISelLowering.h
  lib/Transforms/Scalar/GlobalMerge.cpp
  lib/Transforms/Scalar/Scalar.cpp
  test/CodeGen/AArch64/global-merge.ll
  test/CodeGen/AArch64/global_merge_1.ll
  test/CodeGen/ARM/global-merge-1.ll
  test/CodeGen/ARM64/global-merge.ll
  test/Transforms/GlobalMerge/AArch64/global-merge-1.ll
  test/Transforms/GlobalMerge/AArch64/global-merge-2.ll
  test/Transforms/GlobalMerge/AArch64/lit.local.cfg
  test/Transforms/GlobalMerge/ARM/arm.ll
  test/Transforms/GlobalMerge/ARM64/arm64.ll
  tools/opt/opt.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3432.9137.patch
Type: text/x-patch
Size: 32251 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140507/52d48320/attachment.bin>


More information about the llvm-commits mailing list