Skip to content

dwarf_extractor.cpp: enable limited support for C++#3540

Merged
wenyongh merged 1 commit intobytecodealliance:mainfrom
yamt:dwarf-cxx
Jun 18, 2024
Merged

dwarf_extractor.cpp: enable limited support for C++#3540
wenyongh merged 1 commit intobytecodealliance:mainfrom
yamt:dwarf-cxx

Conversation

@yamt
Copy link
Copy Markdown
Contributor

@yamt yamt commented Jun 18, 2024

While band-aid fixes like this is not plausible IMO, some people prefer to have some debug info even if it's partial/limited/broken. this commit partially (re)enables C++ processing. on the other hand, do not bother to process variables because it's known incompatible with C++.

While band-aid fixes like this is not plausible IMO,
some people prefer to have some debug info even if it's
partial/limited/broken. this commit partially (re)enables
C++ processing. on the other hand, do not bother to process
variables because it's known incompatible with C++.
@TianlongLiang
Copy link
Copy Markdown
Contributor

LGTM

@wenyongh wenyongh merged commit 72f74b7 into bytecodealliance:main Jun 18, 2024
WenLY1 pushed a commit to WenLY1/wasm-micro-runtime that referenced this pull request Jun 26, 2024
…#3540)

While band-aid fixes like this is not plausible IMO,
some people prefer to have some debug info even if it's
partial/limited/broken. This commit partially (re)enables
C++ processing. On the other hand, do not bother to process
variables because it's known incompatible with C++.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants