[PATCH] D89623: [MemCpyOpt] Move GEP during call slot optimization

[dmajor via Phabricator via llvm-commits]

dmajor added a comment.

I thiiiink I'm starting to see this a little better, and the problem is that the memcpy clobbers the optimized call slot.

Before performCallSlotOptzn

  call func(..., X, ...)
  load from X
  GEP
  store to Y

After performCallSlotOptzn

  GEP
  call func(..., Y, ...)
  load from X
  store to Y  ; oops

Am I interpreting that correctly?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89623