From 4ceea0a5055977d43b8277b4767ecaefb90a8e6c Mon Sep 17 00:00:00 2001 From: Craig Topper Date: Sun, 3 Jan 2016 08:57:41 +0000 Subject: [TableGen] Simplify some code slightly. No need to check if the arrays are empty before printing. The loop can be made to print the same thing if the loop is empty. NFC llvm-svn: 256703 --- llvm/utils/TableGen/SubtargetEmitter.cpp | 28 ++++++++++------------------ 1 file changed, 10 insertions(+), 18 deletions(-) (limited to 'llvm') diff --git a/llvm/utils/TableGen/SubtargetEmitter.cpp b/llvm/utils/TableGen/SubtargetEmitter.cpp index 6246d811123..d056de003e1 100644 --- a/llvm/utils/TableGen/SubtargetEmitter.cpp +++ b/llvm/utils/TableGen/SubtargetEmitter.cpp @@ -185,16 +185,12 @@ unsigned SubtargetEmitter::FeatureKeyValues(raw_ostream &OS) { const std::vector &ImpliesList = Feature->getValueAsListOfDefs("Implies"); - if (ImpliesList.empty()) { - OS << "{ }"; - } else { - OS << "{ "; - for (unsigned j = 0, M = ImpliesList.size(); j < M;) { - OS << Target << "::" << ImpliesList[j]->getName(); - if (++j < M) OS << ", "; - } - OS << " }"; + OS << "{"; + for (unsigned j = 0, M = ImpliesList.size(); j < M;) { + OS << " " << Target << "::" << ImpliesList[j]->getName(); + if (++j < M) OS << ","; } + OS << " }"; OS << " }"; ++NumFeatures; @@ -240,16 +236,12 @@ unsigned SubtargetEmitter::CPUKeyValues(raw_ostream &OS) { << "\"" << Name << "\", " << "\"Select the " << Name << " processor\", "; - if (FeatureList.empty()) { - OS << "{ }"; - } else { - OS << "{ "; - for (unsigned j = 0, M = FeatureList.size(); j < M;) { - OS << Target << "::" << FeatureList[j]->getName(); - if (++j < M) OS << ", "; - } - OS << " }"; + OS << "{"; + for (unsigned j = 0, M = FeatureList.size(); j < M;) { + OS << " " << Target << "::" << FeatureList[j]->getName(); + if (++j < M) OS << ","; } + OS << " }"; // The { } is for the "implies" section of this data structure. OS << ", { } }"; -- cgit v1.2.3