[PATCH] D143437: [llvm] Use pointer index type for more GEP offsets (pre-codegen)

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 16 14:59:17 PST 2023


jrtc27 added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/StraightLineStrengthReduce.cpp:677
   case Candidate::GEP:
     {
+    Type *OffsetTy = DL->getIndexType(C.Ins->getType());
----------------
krzysz00 wrote:
> jrtc27 wrote:
> > If you're changing the indentation below please also fix the curlies (and break) to match
> I don't think this is changing the indentation?
Phabricator seems to think you've de-indented by 2 spaces


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143437



More information about the llvm-commits mailing list