Skip to content

Commit

Permalink
[lldb] Don't double emit option groups
Browse files Browse the repository at this point in the history
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.

llvm-svn: 366414
  • Loading branch information
Teemperor committed Jul 18, 2019
1 parent 6371911 commit ce8df1f
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lldb/utils/TableGen/LLDBOptionDefEmitter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@ static void emitOption(Record *Option, raw_ostream &OS) {
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");
Expand Down

0 comments on commit ce8df1f

Please sign in to comment.