[PATCH] D131354: [lld/mac] Use C++17 nested namespace syntax in most places

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 7 07:40:04 PDT 2022


thakis created this revision.
thakis added a reviewer: lld-macho.
Herald added subscribers: ormris, steven_wu, hiraditya.
Herald added projects: lld-macho, All.
thakis requested review of this revision.

Some header files used

  namespace lld {
  namespace macho {
  // ...
  } // namespace macho
  std::string toString(const Type &t);
  } // namespace lld

In those files, I didn't use a nested namespace since it's not a big win there.

No behavior change.


https://reviews.llvm.org/D131354

Files:
  lld/MachO/Arch/ARM64Common.h
  lld/MachO/ConcatOutputSection.h
  lld/MachO/Driver.h
  lld/MachO/Dwarf.h
  lld/MachO/EhFrame.h
  lld/MachO/ExportTrie.h
  lld/MachO/ICF.h
  lld/MachO/LTO.h
  lld/MachO/MachOStructs.h
  lld/MachO/MapFile.h
  lld/MachO/MarkLive.cpp
  lld/MachO/MarkLive.h
  lld/MachO/ObjC.h
  lld/MachO/OutputSection.h
  lld/MachO/OutputSegment.h
  lld/MachO/Relocations.h
  lld/MachO/SectionPriorities.h
  lld/MachO/SymbolTable.h
  lld/MachO/SyntheticSections.h
  lld/MachO/Target.h
  lld/MachO/UnwindInfoSection.h
  lld/MachO/Writer.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D131354.450624.patch
Type: text/x-patch
Size: 10421 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220807/d56ba81f/attachment.bin>


More information about the llvm-commits mailing list