Like D131354, but for COFF.
No behavior change.
Paths
| Differential D131405
[lld/win] Use C++17 nested namespace syntax in most places ClosedPublic Authored by thakis on Aug 8 2022, 8:35 AM.
Details
Diff Detail
Event TimelineComment Actions Looks reasonable to me. I guess the coding style guidelines will be updated for C++17 features eventually (about what features to use and not use) - this particular change seems uncontroversial in any case. This revision is now accepted and ready to land.Aug 8 2022, 8:45 AM This revision was landed with ongoing or failed builds.Aug 8 2022, 8:48 AM Closed by commit rG7c26641d9dce: [lld/win] Use C++17 nested namespace syntax in most places (authored by thakis). · Explain Why This revision was automatically updated to reflect the committed changes. Comment Actions In general, when making these kind of generic changes, would it be possible to also include wasm-ld (lld/wasm) so that we keep things as in-sync as possible?
Revision Contents
Diff 450830 lld/COFF/COFFLinkerContext.h
lld/COFF/COFFLinkerContext.cpp
lld/COFF/CallGraphSort.h
lld/COFF/Chunks.h
lld/COFF/Chunks.cpp
lld/COFF/Config.h
lld/COFF/DLL.h
lld/COFF/DLL.cpp
lld/COFF/DebugTypes.h
lld/COFF/DebugTypes.cpp
lld/COFF/Driver.h
lld/COFF/Driver.cpp
lld/COFF/ICF.h
lld/COFF/ICF.cpp
lld/COFF/LLDMapFile.h
lld/COFF/LTO.h
lld/COFF/MapFile.h
lld/COFF/MarkLive.h
lld/COFF/MarkLive.cpp
lld/COFF/MinGW.h
lld/COFF/PDB.h
lld/COFF/SymbolTable.h
lld/COFF/SymbolTable.cpp
lld/COFF/TypeMerger.h
lld/COFF/Writer.h
|