[PATCH] D43405: [WebAssembly] Remove unneeded Chunk::getFileName() method. NFC.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 15:49:57 PST 2018


sbc100 added a comment.

In https://reviews.llvm.org/D43405#1010999, @ruiu wrote:

> LGTM
>
> But I believe a better way of doing this kind of stuff is to define toString(InputChunk *). toString() function is a common interface to stringize objects to create debug messages.


But in this case the only user actually wants the filename on its own.. not the string description of the chunk.    (ELF does basically exactly the same thing here BTW)


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D43405





More information about the llvm-commits mailing list