[PATCH] D125511: [InstCombine] Merging constant-indexed GEP of GEP in both directions

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat May 14 01:37:25 PDT 2022


nikic added a comment.

@huangjd The not all constant case continues to be handled. If you mean the case where we have `((p + C) + x), then this should first be converted to `((p + x) + C` by the previously discussed transform to hoist constant GEPs, at which point the constant GEP is in the right position of this transform again. There is no need to additionally perform a swap at this point.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125511



More information about the llvm-commits mailing list