[PATCH] D125070: [InstCombine] Simplify chain of GEP with constant indices

William Junda Huang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 10 19:24:42 PDT 2022


huangjd added a comment.

@nikic

> 2. Combine constant GEPs. We already do this, but sometimes fail based on element type mismatch. This can be fixed either by canonicalizing all constant GEPs to `i8` (probably better) or by doing so only when it allows combining GEPs that otherwise couldn't be combined.

Where are the tests located for combining constant GEP?  I think currently it only work with one direction (merging the second one into the first), so I will need to make it work both ways.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125070



More information about the llvm-commits mailing list