Change Details

After we landed https://reviews.llvm.org/D127383, the compiler may crash at: https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp#L687-L691. This is because the dbg intrinsics are readnone originally and the MemorySSA would refuse to create a new memory access at https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Analysis/MemorySSA.cpp#L1830-L1831 But after D127383, it wouldn't be true for the dbg intrinsics in presplit coroutine. So here is the crash.

After we landed https://reviews.llvm.org/D127383, the compiler may crash at: https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp#L687-L691. This is because the dbg intrinsics are readnone originally and the MemorySSA would refuse to create a new memory access at https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Analysis/MemorySSA.cpp#L1830-L1831 But after D127383, it wouldn't be true for the dbg intrinsics in presplit coroutine. So here is the crash. This would be a NFC patch before D127383 landed.

After we landed https://reviews.llvm.org/D127383, the compiler may crash at: https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp#L687-L691. This is because the dbg intrinsics are readnone originally and the MemorySSA would refuse to create a new memory access at https://github.com/llvm/llvm-project/blob/051738b08cf5e39fd274dd379147d1c19e2b5b20/llvm/lib/Analysis/MemorySSA.cpp#L1830-L1831 But after D127383, it wouldn't be true for the dbg intrinsics in presplit coroutine. So here is the crash. This would be a NFC patch before D127383 landed.