[PATCH] D81055: [WebAssembly] Lower llvm.debugtrap properly

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added a comment.

I'm not very familiar with how wasm debugger works, but do they use `unreachable` for breakpoints? Binaryen assumes `unreachable` is really unreachable and does all sorts of optimizations including DCE and everything, so if `unreachable` is actually used in debugger, that seems risky..?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D81055