[all-commits] [llvm/llvm-project] 144ec1: [WebAssembly] Encode numbers in ULEB128 in event s...

Heejin Ahn via All-commits all-commits at lists.llvm.org
Tue Mar 30 16:22:24 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 144ec1c38ef1d3a6d37dadf7240ef5f04c10daae
      https://github.com/llvm/llvm-project/commit/144ec1c38ef1d3a6d37dadf7240ef5f04c10daae
  Author: Heejin Ahn <aheejin at gmail.com>
  Date:   2021-03-30 (Tue, 30 Mar 2021)

  Changed paths:
    M llvm/lib/Object/WasmObjectFile.cpp
    A llvm/test/MC/WebAssembly/event-section-encoding.ll

  Log Message:
  -----------
  [WebAssembly] Encode numbers in ULEB128 in event section

The number of events and the type index should be encoded in ULEB128,
but they were incorrctly encoded in LEB128. The smallest number with
which its LEB128 and ULEB128 encodings are different is 64.
There's no way we can generate 64 events in the C++ toolchain
implementation so we can't test that, but the attached test tests when
the type index is 64.

Reviewed By: dschuff

Differential Revision: https://reviews.llvm.org/D99627




More information about the All-commits mailing list