[PATCH] D123903: [IRSim] Ignore Debug instructions with constructing canonical numbering for basic blocks

[Bjorn Pettersson via Phabricator via llvm-commits]

bjope added a comment.

Thanks for the patch. I've verified that it solves the problem I found.

No comments about the patch itself really. Looks nice, but I don't know that much about this analysis pass to say if it is correct to skip dbg intrinsics like this. (And should for example lifetime instrinsics be skipped in the same way?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123903