[PATCH] D54924: [WebAssembly] Check if the section order is correct

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 27 16:53:38 PST 2018


aheejin added a comment.

I see.. so why wasn't it landed? It says it was stalled because we couldn't decide whether or how to check custom section orderings and orderings between custom section vs. normal sections. Do we now have consensus on that? If so landing that CL makes more sense. If not, do you think now is a good time to re-discuss it? Or would landing this CL first and decide what to do with custom sections later makes sense?

The main motivation for this CL was, I had to delete code that was previously checking relative section orders in `yaml2wasm` because the previous code only checked if the section codes were monotonically increasing and the new event section had code 12, and you suggested <https://reviews.llvm.org/D54096#1295722> we should reinstate it in a separate CL.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54924/new/

https://reviews.llvm.org/D54924





More information about the llvm-commits mailing list