[PATCH] D51462: [ARM] Enable GEP offset splitting for 32-bit ARM.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 29 15:42:15 PDT 2018


efriedma created this revision.
efriedma added reviewers: samparker, dmgreen, t.p.northover.
Herald added a reviewer: javed.absar.
Herald added subscribers: chrib, kristof.beyls.

It has essentially the same benefit it has on 64-bit ARM: it substantially reduces the number of constants used by large GEP operations. Seems to be generally helpful across a few different codebases I've tried.

It is a bit heuristic driven, though, so I'm not sure it's universally beneficial; maybe could use more testing.


Repository:
  rL LLVM

https://reviews.llvm.org/D51462

Files:
  lib/Target/ARM/ARMISelLowering.h
  test/Transforms/CodeGenPrepare/ARM/large-offset-gep.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51462.163203.patch
Type: text/x-patch
Size: 4987 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180829/883be6d6/attachment.bin>


More information about the llvm-commits mailing list