Skip to content

Commit 9e0d550

Browse files
committed
[clang][NFC] Clean up whitespace in ClangOpcodesEmitter output
This should now be about as style-conforming as TableGen'ed code ever can reasonably be.
1 parent a0c1cd6 commit 9e0d550

File tree

1 file changed

+26
-25
lines changed

1 file changed

+26
-25
lines changed

clang/utils/TableGen/ClangOpcodesEmitter.cpp

Lines changed: 26 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -122,13 +122,13 @@ void ClangOpcodesEmitter::EmitInterp(raw_ostream &OS, StringRef N, Record *R) {
122122

123123
// Emit calls to read arguments.
124124
for (size_t I = 0, N = Args.size(); I < N; ++I) {
125-
OS << "\tauto V" << I;
125+
OS << " auto V" << I;
126126
OS << " = ";
127127
OS << "PC.read<" << Args[I]->getValueAsString("Name") << ">();\n";
128128
}
129129

130130
// Emit a call to the template method and pass arguments.
131-
OS << "\tif (!" << N;
131+
OS << " if (!" << N;
132132
PrintTypes(OS, TS);
133133
OS << "(S";
134134
if (ChangesPC)
@@ -140,15 +140,15 @@ void ClangOpcodesEmitter::EmitInterp(raw_ostream &OS, StringRef N, Record *R) {
140140
for (size_t I = 0, N = Args.size(); I < N; ++I)
141141
OS << ", V" << I;
142142
OS << "))\n";
143-
OS << "\t\treturn false;\n";
143+
OS << " return false;\n";
144144

145145
// Bail out if interpreter returned.
146146
if (CanReturn) {
147-
OS << "\tif (!S.Current || S.Current->isRoot())\n";
148-
OS << "\t\treturn true;\n";
147+
OS << " if (!S.Current || S.Current->isRoot())\n";
148+
OS << " return true;\n";
149149
}
150150

151-
OS << "\tcontinue;\n";
151+
OS << " continue;\n";
152152
OS << "}\n";
153153
});
154154
OS << "#endif\n";
@@ -158,14 +158,14 @@ void ClangOpcodesEmitter::EmitDisasm(raw_ostream &OS, StringRef N, Record *R) {
158158
OS << "#ifdef GET_DISASM\n";
159159
Enumerate(R, N, [R, &OS](ArrayRef<Record *>, const Twine &ID) {
160160
OS << "case OP_" << ID << ":\n";
161-
OS << "\tPrintName(\"" << ID << "\");\n";
162-
OS << "\tOS << \"\\t\"";
161+
OS << " PrintName(\"" << ID << "\");\n";
162+
OS << " OS << \"\\t\"";
163163

164164
for (auto *Arg : R->getValueAsListOfDefs("Args"))
165165
OS << " << PC.read<" << Arg->getValueAsString("Name") << ">() << \" \"";
166166

167-
OS << "<< \"\\n\";\n";
168-
OS << "\tcontinue;\n";
167+
OS << " << \"\\n\";\n";
168+
OS << " continue;\n";
169169
});
170170
OS << "#endif\n";
171171
}
@@ -181,11 +181,11 @@ void ClangOpcodesEmitter::EmitEmitter(raw_ostream &OS, StringRef N, Record *R) {
181181
// Emit the list of arguments.
182182
OS << "bool ByteCodeEmitter::emit" << ID << "(";
183183
for (size_t I = 0, N = Args.size(); I < N; ++I)
184-
OS << Args[I]->getValueAsString("Name") << " A" << I << ",";
184+
OS << Args[I]->getValueAsString("Name") << " A" << I << ", ";
185185
OS << "const SourceInfo &L) {\n";
186186

187187
// Emit a call to write the opcodes.
188-
OS << "\treturn emitOp<";
188+
OS << " return emitOp<";
189189
for (size_t I = 0, N = Args.size(); I < N; ++I) {
190190
if (I != 0)
191191
OS << ", ";
@@ -250,7 +250,7 @@ void ClangOpcodesEmitter::EmitGroup(raw_ostream &OS, StringRef N, Record *R) {
250250

251251
// Emit the dispatch implementation in the source.
252252
OS << "#if defined(GET_EVAL_IMPL) || defined(GET_LINK_IMPL)\n";
253-
OS << "bool \n";
253+
OS << "bool\n";
254254
OS << "#if defined(GET_EVAL_IMPL)\n";
255255
OS << "EvalEmitter\n";
256256
OS << "#else\n";
@@ -271,13 +271,14 @@ void ClangOpcodesEmitter::EmitGroup(raw_ostream &OS, StringRef N, Record *R) {
271271
// Custom evaluator methods dispatch to template methods.
272272
if (R->getValueAsBit("HasCustomEval")) {
273273
OS << "#ifdef GET_LINK_IMPL\n";
274-
OS << "return emit" << ID << "\n";
274+
OS << " return emit" << ID << "\n";
275275
OS << "#else\n";
276-
OS << "return emit" << N;
276+
OS << " return emit" << N;
277277
PrintTypes(OS, TS);
278278
OS << "\n#endif\n";
279+
OS << " ";
279280
} else {
280-
OS << "return emit" << ID;
281+
OS << " return emit" << ID;
281282
}
282283

283284
OS << "(";
@@ -290,19 +291,19 @@ void ClangOpcodesEmitter::EmitGroup(raw_ostream &OS, StringRef N, Record *R) {
290291

291292
// Print a switch statement selecting T.
292293
if (auto *TypeClass = dyn_cast<DefInit>(Types->getElement(I))) {
293-
OS << "switch (T" << I << "){\n";
294+
OS << " switch (T" << I << ") {\n";
294295
auto Cases = TypeClass->getDef()->getValueAsListOfDefs("Types");
295296
for (auto *Case : Cases) {
296-
OS << "case PT_" << Case->getName() << ":\n";
297+
OS << " case PT_" << Case->getName() << ":\n";
297298
TS.push_back(Case);
298299
Rec(I + 1, ID + Case->getName());
299300
TS.pop_back();
300301
}
301302
// Emit a default case if not all types are present.
302303
if (Cases.size() < NumTypes)
303-
OS << "default: llvm_unreachable(\"invalid type\");\n";
304-
OS << "}\n";
305-
OS << "llvm_unreachable(\"invalid enum value\");\n";
304+
OS << " default: llvm_unreachable(\"invalid type\");\n";
305+
OS << " }\n";
306+
OS << " llvm_unreachable(\"invalid enum value\");\n";
306307
} else {
307308
PrintFatalError("Expected a type class");
308309
}
@@ -323,12 +324,12 @@ void ClangOpcodesEmitter::EmitEval(raw_ostream &OS, StringRef N, Record *R) {
323324

324325
OS << "bool EvalEmitter::emit" << ID << "(";
325326
for (size_t I = 0, N = Args.size(); I < N; ++I)
326-
OS << Args[I]->getValueAsString("Name") << " A" << I << ",";
327+
OS << Args[I]->getValueAsString("Name") << " A" << I << ", ";
327328
OS << "const SourceInfo &L) {\n";
328-
OS << "if (!isActive()) return true;\n";
329-
OS << "CurrentSource = L;\n";
329+
OS << " if (!isActive()) return true;\n";
330+
OS << " CurrentSource = L;\n";
330331

331-
OS << "return " << N;
332+
OS << " return " << N;
332333
PrintTypes(OS, TS);
333334
OS << "(S, OpPC";
334335
for (size_t I = 0, N = Args.size(); I < N; ++I)

0 commit comments

Comments
 (0)