This is an archive of the discontinued LLVM Phabricator instance.

[clang][Driver] use DWARF4 for wasm
ClosedPublic

Authored by dschuff on Jan 24 2022, 3:00 PM.

Details

Summary

Opt into the old default of DWARF4 for now.

Diff Detail

Event Timeline

dschuff created this revision.Jan 24 2022, 3:00 PM
dschuff requested review of this revision.Jan 24 2022, 3:01 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 24 2022, 3:01 PM

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).

MaskRay added inline comments.
clang/test/Driver/debug-options.c
141

Keep just one blank line

dschuff updated this revision to Diff 402694.Jan 24 2022, 3:26 PM

just one blank line

aardappel accepted this revision.Jan 24 2022, 3:28 PM
This revision is now accepted and ready to land.Jan 24 2022, 3:28 PM
This revision was landed with ongoing or failed builds.Jan 24 2022, 3:48 PM
This revision was automatically updated to reflect the committed changes.