[PATCH] D54096: [WebAssembly] Add support for the event section

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 6 19:56:27 PST 2018


aheejin added a comment.

In https://reviews.llvm.org/D54096#1287080, @aheejin wrote:

> - Currently `test/Object/wasm-string-outside-section.test` fails with this patch. This test case is testing a malformed custom section, and I have to update the test case binary to include the new event section to maintain the same failure behavior (otherwise this fails before it reaches the custom section). This test was added in https://reviews.llvm.org/D50387 and I don't know how this test binary was generated. Asked the author about it.


This is now passing. This failed because I incorrectly changed the section codes in my first patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D54096





More information about the llvm-commits mailing list