[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