[PATCH] [Transforms/Scalar/SeparateConstOffsetFromGEP] Add the extracted offset using GEP

Jingyue Wu jingyue at google.com
Thu May 22 15:56:53 PDT 2014


Hi eliben, hfinkel,

Fixed a TODO in the original code. 

We add the extracted constant offset using GEP instead of ugly ptrtoint+add+inttoptr. Using GEP simplifies future optimizations and makes IR easier to understand. 

Updated all affected tests, and added a new test in split-gep.ll to cover a corner case where uglygep is necessary.

http://reviews.llvm.org/D3882

Files:
  lib/Transforms/Scalar/SeparateConstOffsetFromGEP.cpp
  test/Transforms/SeparateConstOffsetFromGEP/NVPTX/split-gep-and-gvn.ll
  test/Transforms/SeparateConstOffsetFromGEP/NVPTX/split-gep.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3882.9720.patch
Type: text/x-patch
Size: 8930 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140522/3a06b946/attachment.bin>


More information about the llvm-commits mailing list