[PATCH] D68323: [ELF] Wrap things in `namespace lld { namespace elf {`, NFC

Thu Oct 3 22:48:03 PDT 2019

ruiu added a comment.

In D68323#1694079 <https://reviews.llvm.org/D68323#1694079>, @MaskRay wrote:

> In D68323#1694050 <https://reviews.llvm.org/D68323#1694050>, @ruiu wrote:
>
> > How about always using elf::write32 instead of endian::write32<E>?
>
>
> `lld::elf::*` are more commonly used in lld. This will end up with a lot of qualified `write32` `read16` calls. I think we should reduce the number of `using namespace llvm::support::endian;`.

I mean we can replace all occurrences of `endian::write32<E>()` with `write32()`. It looks like lld still compiles after I run the following command to replace them.

sed -i 's/write32<.>/write32/g' lld/ELF/Arch/Mips.cpp

Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D68323