[PATCH] D42759: [CGP] Split large data structres to sink more GEPs

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 20 15:41:14 PDT 2022


efriedma added inline comments.


================
Comment at: llvm/trunk/lib/CodeGen/CodeGenPrepare.cpp:3816
+                 TLI.shouldConsiderGEPOffsetSplit() && Depth == 0 &&
+                 ConstantOffset > 0) {
+        // Record GEPs with non-zero offsets as candidates for splitting in the
----------------
jonpa wrote:
> Is there any reason to not use '!=' as oppsed to '>' here (which would match the comment below)?
> 
> On SystemZ negative offsets are not supported on vector instructions.
> 
> 
> 
It could work. Would probably need a few other changes to make sure we don't do anything silly like grouping together positive and negative offsets.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D42759/new/

https://reviews.llvm.org/D42759



More information about the llvm-commits mailing list