[Lldb-commits] [PATCH] D65331: [lldb] Also include the array definition in CommandOptions.inc

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jul 26 13:28:00 PDT 2019


JDevlieghere added inline comments.


================
Comment at: lldb/utils/TableGen/LLDBOptionDefEmitter.cpp:146
   // We undefine the macro for the user like Clang's include files are doing it.
+  OS << "};\n";
   OS << "#undef " << NeededMacro << "\n";
----------------
Let's move this before the comment.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65331/new/

https://reviews.llvm.org/D65331





More information about the lldb-commits mailing list