Bug Summary

File:tools/lld/COFF/Config.h
Warning:line 75, column 8
Excessive padding in 'struct lld::coff::Configuration' (48 padding bytes, where 0 is optimal). Optimal fields order: Entry, DelayLoadHelper, SEHTable, SEHCount, ImageBase, StackReserve, StackCommit, HeapReserve, HeapCommit, Implib, LTOCache, ManifestDependency, ManifestLevel, ManifestUIAccess, ManifestFile, Argv, GCRoot, Exports, ManifestInput, OutputFile, ImportName, MapFile, NoDefaultLibs, DelayLoads, DLLOrder, LTOCachePolicy, Merge, Section, AlignComm, MustMatch, AlternateNames, PDBPath, Machine, Subsystem, DebugTypes, LTOOptLevel, LTOJobs, LTOPartitions, Manifest, ManifestID, MajorImageVersion, MinorImageVersion, MajorOSVersion, MinorOSVersion, Verbose, NoEntry, DoGC, DoICF, Relocatable, Force, Debug, DebugDwarf, DebugGHashes, NoDefaultLibAll, DLL, SaveTemps, ManifestUAC, CanExitEarly, DynamicBase, AllowBind, NxCompat, AllowIsolation, TerminalServerAware, LargeAddressAware, HighEntropyVA, AppContainer, MinGW, WarnLocallyDefinedImported, consider reordering the fields or adding explicit padding members

Annotated Source Code

[?] Use j/k keys for keyboard navigation

1//===- Config.h -------------------------------------------------*- C++ -*-===//
2//
3// The LLVM Linker
4//
5// This file is distributed under the University of Illinois Open Source
6// License. See LICENSE.TXT for details.
7//
8//===----------------------------------------------------------------------===//
9
10#ifndef LLD_COFF_CONFIG_H
11#define LLD_COFF_CONFIG_H
12
13#include "llvm/ADT/StringRef.h"
14#include "llvm/Object/COFF.h"
15#include "llvm/Support/CachePruning.h"
16#include <cstdint>
17#include <map>
18#include <set>
19#include <string>
20
21namespace lld {
22namespace coff {
23
24using llvm::COFF::IMAGE_FILE_MACHINE_UNKNOWN;
25using llvm::COFF::WindowsSubsystem;
26using llvm::StringRef;
27class DefinedAbsolute;
28class DefinedRelative;
29class StringChunk;
30class Symbol;
31
32// Short aliases.
33static const auto AMD64 = llvm::COFF::IMAGE_FILE_MACHINE_AMD64;
34static const auto ARM64 = llvm::COFF::IMAGE_FILE_MACHINE_ARM64;
35static const auto ARMNT = llvm::COFF::IMAGE_FILE_MACHINE_ARMNT;
36static const auto I386 = llvm::COFF::IMAGE_FILE_MACHINE_I386;
37
38// Represents an /export option.
39struct Export {
40 StringRef Name; // N in /export:N or /export:E=N
41 StringRef ExtName; // E in /export:E=N
42 Symbol *Sym = nullptr;
43 uint16_t Ordinal = 0;
44 bool Noname = false;
45 bool Data = false;
46 bool Private = false;
47 bool Constant = false;
48
49 // If an export is a form of /export:foo=dllname.bar, that means
50 // that foo should be exported as an alias to bar in the DLL.
51 // ForwardTo is set to "dllname.bar" part. Usually empty.
52 StringRef ForwardTo;
53 StringChunk *ForwardChunk = nullptr;
54
55 // True if this /export option was in .drectves section.
56 bool Directives = false;
57 StringRef SymbolName;
58 StringRef ExportName; // Name in DLL
59
60 bool operator==(const Export &E) {
61 return (Name == E.Name && ExtName == E.ExtName &&
62 Ordinal == E.Ordinal && Noname == E.Noname &&
63 Data == E.Data && Private == E.Private);
64 }
65};
66
67enum class DebugType {
68 None = 0x0,
69 CV = 0x1, /// CodeView
70 PData = 0x2, /// Procedure Data
71 Fixup = 0x4, /// Relocation Table
72};
73
74// Global configuration.
75struct Configuration {
Excessive padding in 'struct lld::coff::Configuration' (48 padding bytes, where 0 is optimal). Optimal fields order: Entry, DelayLoadHelper, SEHTable, SEHCount, ImageBase, StackReserve, StackCommit, HeapReserve, HeapCommit, Implib, LTOCache, ManifestDependency, ManifestLevel, ManifestUIAccess, ManifestFile, Argv, GCRoot, Exports, ManifestInput, OutputFile, ImportName, MapFile, NoDefaultLibs, DelayLoads, DLLOrder, LTOCachePolicy, Merge, Section, AlignComm, MustMatch, AlternateNames, PDBPath, Machine, Subsystem, DebugTypes, LTOOptLevel, LTOJobs, LTOPartitions, Manifest, ManifestID, MajorImageVersion, MinorImageVersion, MajorOSVersion, MinorOSVersion, Verbose, NoEntry, DoGC, DoICF, Relocatable, Force, Debug, DebugDwarf, DebugGHashes, NoDefaultLibAll, DLL, SaveTemps, ManifestUAC, CanExitEarly, DynamicBase, AllowBind, NxCompat, AllowIsolation, TerminalServerAware, LargeAddressAware, HighEntropyVA, AppContainer, MinGW, WarnLocallyDefinedImported, consider reordering the fields or adding explicit padding members
76 enum ManifestKind { SideBySide, Embed, No };
77 bool is64() { return Machine == AMD64 || Machine == ARM64; }
78
79 llvm::COFF::MachineTypes Machine = IMAGE_FILE_MACHINE_UNKNOWN;
80 bool Verbose = false;
81 WindowsSubsystem Subsystem = llvm::COFF::IMAGE_SUBSYSTEM_UNKNOWN;
82 Symbol *Entry = nullptr;
83 bool NoEntry = false;
84 std::string OutputFile;
85 std::string ImportName;
86 bool DoGC = true;
87 bool DoICF = true;
88 bool Relocatable = true;
89 bool Force = false;
90 bool Debug = false;
91 bool DebugDwarf = false;
92 bool DebugGHashes = false;
93 unsigned DebugTypes = static_cast<unsigned>(DebugType::None);
94 llvm::SmallString<128> PDBPath;
95 std::vector<llvm::StringRef> Argv;
96
97 // Symbols in this set are considered as live by the garbage collector.
98 std::vector<Symbol *> GCRoot;
99
100 std::set<StringRef> NoDefaultLibs;
101 bool NoDefaultLibAll = false;
102
103 // True if we are creating a DLL.
104 bool DLL = false;
105 StringRef Implib;
106 std::vector<Export> Exports;
107 std::set<std::string> DelayLoads;
108 std::map<std::string, int> DLLOrder;
109 Symbol *DelayLoadHelper = nullptr;
110
111 bool SaveTemps = false;
112
113 // Used for SafeSEH.
114 Symbol *SEHTable = nullptr;
115 Symbol *SEHCount = nullptr;
116
117 // Used for /opt:lldlto=N
118 unsigned LTOOptLevel = 2;
119
120 // Used for /opt:lldltojobs=N
121 unsigned LTOJobs = 0;
122 // Used for /opt:lldltopartitions=N
123 unsigned LTOPartitions = 1;
124
125 // Used for /opt:lldltocache=path
126 StringRef LTOCache;
127 // Used for /opt:lldltocachepolicy=policy
128 llvm::CachePruningPolicy LTOCachePolicy;
129
130 // Used for /merge:from=to (e.g. /merge:.rdata=.text)
131 std::map<StringRef, StringRef> Merge;
132
133 // Used for /section=.name,{DEKPRSW} to set section attributes.
134 std::map<StringRef, uint32_t> Section;
135
136 // Options for manifest files.
137 ManifestKind Manifest = No;
138 int ManifestID = 1;
139 StringRef ManifestDependency;
140 bool ManifestUAC = true;
141 std::vector<std::string> ManifestInput;
142 StringRef ManifestLevel = "'asInvoker'";
143 StringRef ManifestUIAccess = "'false'";
144 StringRef ManifestFile;
145
146 // Used for /aligncomm.
147 std::map<std::string, int> AlignComm;
148
149 // Used for /failifmismatch.
150 std::map<StringRef, StringRef> MustMatch;
151
152 // Used for /alternatename.
153 std::map<StringRef, StringRef> AlternateNames;
154
155 // Used for /lldmap.
156 std::string MapFile;
157
158 uint64_t ImageBase = -1;
159 uint64_t StackReserve = 1024 * 1024;
160 uint64_t StackCommit = 4096;
161 uint64_t HeapReserve = 1024 * 1024;
162 uint64_t HeapCommit = 4096;
163 uint32_t MajorImageVersion = 0;
164 uint32_t MinorImageVersion = 0;
165 uint32_t MajorOSVersion = 6;
166 uint32_t MinorOSVersion = 0;
167 bool CanExitEarly = false;
168 bool DynamicBase = true;
169 bool AllowBind = true;
170 bool NxCompat = true;
171 bool AllowIsolation = true;
172 bool TerminalServerAware = true;
173 bool LargeAddressAware = false;
174 bool HighEntropyVA = false;
175 bool AppContainer = false;
176 bool MinGW = false;
177 bool WarnLocallyDefinedImported = true;
178};
179
180extern Configuration *Config;
181
182} // namespace coff
183} // namespace lld
184
185#endif