[PATCH] D119886: [AMDGPU] Promote recursive loads from kernel argument to constant

[Stanislav Mekhanoshin via Phabricator via llvm-commits]

rampitec added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUPromoteKernelArguments.cpp:138
+  Value *OrigPtr = Ptr;
+  while (isa<AddrSpaceCastInst>(OrigPtr))
+    OrigPtr = cast<AddrSpaceCastInst>(OrigPtr)->getPointerOperand();
----------------
arsenm wrote:
> rampitec wrote:
> > vpykhtin wrote:
> > > Nit: you can shorten the loop unsing while (auto *P = dyn_cast<AddrSpaceCastInst>(OrigPtr)) ...
> > I need both Ptr and OrigPtr after the loop.
> stripPointerCasts?
That's too much, I only need to strip address space casts. stripPointerCasts may change type.


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

https://reviews.llvm.org/D119886