[Lldb-commits] [lldb] r366414 - [lldb] Don't double emit option groups

[Raphael Isemann via lldb-commits]

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");