[PATCH] D118082: [clang][Driver] use DWARF4 for wasm

[Derek Schuff via Phabricator via cfe-commits]

dschuff added subscribers: pfaffe, azakai.
dschuff added a comment.

Also +cc @pfaffe and @azakai

I'd like to land this soon to fix our emscripten tests for now. But we should talk about what the default should be. I'm curious if @sunfish has any any opinions on DWARF for their use cases and AFAIK lldb/chrome devtools is (or is almost?) ready for DWARF5 so maybe we can go back to defaulting to dwarf5 once we figure out exactly what to do for symbolizing stack traces of optimized builds (since Binaryen doesn't support DWARF5 yet, and may or may not soon).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118082