[PATCH] D152659: [LV] Add IRBuilder folder to simplify GEP x, 0.

Ayal Zaks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 15 07:17:46 PDT 2023


Ayal added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:9751
       Value *Increment = createStepForVF(Builder, IndexTy, State.VF, Part);
       PartPtr = Builder.CreateGEP(ScalarDataTy, Ptr, Increment, "", InBounds);
     }
----------------
Is this (the only place) where the redundant gep-with-zero are generated?

Would be good to ensure recipes are (more) aware of the code they generate, and potentially account for it cost-wise, rather than folding this inside IRBuilder?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152659



More information about the llvm-commits mailing list