[PATCH] D126277: [Debug] [Coroutines] Add deref operator for non complex expression

[Chuanqi Xu via Phabricator via llvm-commits]

ChuanqiXu added a comment.

In D126277#3562899 <https://reviews.llvm.org/D126277#3562899>, @jmorse wrote:

> (All away for the UK holiday week sorry),
>
> Thanks for the detail on how salvaging happens with coroutines -- I was curious why it would depend on what was already in the DIExpression. The deref is indeed necessary because the pointer is in an alloca. I suspect the check for `isComplex()` wasn't completely necessary (but only because of DIExpression quirks, unfortunately).
>
> LGTM, but could you put an extra CHECK in where I've marked to capture FramePtr_RESUME using the alloca instruction. When reading the test, it wasn't clear to me where the dbg.declare was going to point, this will make it easier for future readers to understand.

Thanks for reviewing!


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

https://reviews.llvm.org/D126277