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

Wed Oct 28 15:03:16 PDT 2020

dmajor added a comment.

Hi, we have a regression that started with this commit. The problem happens when GEP %43 is moved before the call on line 49: https://paste.mozilla.org/gYqBnKes . Can you see anything obviously wrong with that move?

If not, our repro is quite large and I fear it will take me a very long time to fully understand what went wrong and/or get a runnable reduced test case. At this point I'm not even sure whether this transformation was itself the culprit or merely uncovered a problem elsewhere. Hoping you might be able to quickly spot something and save me lots of time!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89623