[PATCH] D46416: [WebAssembly] Add option to remove LEB padding at relocation sites

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 7 18:55:20 PDT 2018


ruiu added a comment.

Overall looking good, but I think I have a concern about the lack of comments. Could you expand comments to make the .cpp file more self-explanatory so that a first-time reader who's not very familiar with wasm can understand the code by reading it?


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D46416





More information about the llvm-commits mailing list