[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