Skip to content

objdump: add support for segments flags in the linking section #1669

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 13, 2021

Conversation

sbc100
Copy link
Member

@sbc100 sbc100 commented May 13, 2021

No description provided.

@sbc100 sbc100 requested a review from dschuff May 13, 2021 01:11
Copy link
Member

@dschuff dschuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do like the improvement in readability too

@sbc100 sbc100 requested a review from dschuff May 13, 2021 21:42
@sbc100 sbc100 enabled auto-merge (squash) May 13, 2021 22:57
@sbc100 sbc100 merged commit 63c3509 into main May 13, 2021
@sbc100 sbc100 deleted the segment_flags branch May 13, 2021 23:01
takikawa added a commit to takikawa/wasm-tools that referenced this pull request Jun 22, 2021
A recent PR to wabt changed the printing slightly for these test cases:

  WebAssembly/wabt#1669
alexcrichton pushed a commit to bytecodealliance/wasm-tools that referenced this pull request Jun 23, 2021
#291)

* Update wabt test submodule to latest

* Update wasm-encoder linking tests for latest wabt

A recent PR to wabt changed the printing slightly for these test cases:

  WebAssembly/wabt#1669

* Rename "events" to "tags" for exception handling

The exception handling proposal spec was recently changed to rename the
event section to the tag section, in order to use a clearer name that
conflicts with fewer things in the web context:

  WebAssembly/exception-handling#161
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.

2 participants