@dstenb Thanks for the review. Any additional comments?

Address comment - delaySlotSupported change.
Compress test-CHECK-lines.

Clang format.

Address comments.
Rebase.

Capture "&" instead of "this" in delaySlotSupported lambda.

Address comments.
Change of delay slot guard - use assertions instead of target check.

Address comments.
Transform lambda

nextInstrInterpret

into static function

interpretNextInstr

Address comment - spelling.
Rebase.

In D78105#2035965, @dstenb wrote:

LGTM. Thanks!

Do you need help with landing this?

Address comments:

• tests changes
• patch summary update

In D78105#2025646, @vsk wrote:

This seems to be in great shape. Does it have any dependencies?

Address comment.
Tests update - use only one !DILocation per scope.

The main purpose of the "delay slot guard" is to ensure that call_site DIE won't be created for call with delay slot, because it could have incorrect "return PC" value (DW_AT_low_pc, DW_AT_call_return_pc).
This will happen if the call and its delay slot instruction are not in the expected order.

Address comment.

Address comments.
@dstenb @djtodoro Thanks for your suggestions.

Address comments.

Merging into https://reviews.llvm.org/D78105.

Address comment.
Add changes from https://reviews.llvm.org/D78104.

Address comments.

@dstenb
Thanks for the review.
I agree with all suggestions.

In D78104#1985262, @aprantl wrote:

This would benefit from some kind of test.