Author: teemperor Date: Thu Jul 18 01:22:11 2019 New Revision: 366414 URL: http://llvm.org/viewvc/llvm-project?rev=366414&view=rev Log: [lldb] Don't double emit option groups
We currently emit the option groups twice if Groups<[1,2,3]> is used in the tablegen. This leads to compilation errors. This patch just removes the line that accidentially emits the option group a second time. Modified: lldb/trunk/utils/TableGen/LLDBOptionDefEmitter.cpp Modified: lldb/trunk/utils/TableGen/LLDBOptionDefEmitter.cpp URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/utils/TableGen/LLDBOptionDefEmitter.cpp?rev=366414&r1=366413&r2=366414&view=diff ============================================================================== --- lldb/trunk/utils/TableGen/LLDBOptionDefEmitter.cpp (original) +++ lldb/trunk/utils/TableGen/LLDBOptionDefEmitter.cpp Thu Jul 18 01:22:11 2019 @@ -44,7 +44,6 @@ static void emitOption(Record *Option, r auto Groups = Option->getValueAsListOfInts("Groups"); for (int Group : Groups) GroupsArg.push_back("LLDB_OPT_SET_" + std::to_string(Group)); - OS << llvm::join(GroupsArg.begin(), GroupsArg.end(), " | "); } else if (Option->getValue("GroupStart")) { // The user specified a range of groups (with potentially only one element). int GroupStart = Option->getValueAsInt("GroupStart"); _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits