[Lldb-commits] [PATCH] D43099: Make LLDB's clang module cache path customizable

Adrian Prantl via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Feb 9 10:02:38 PST 2018

aprantl added inline comments.

Comment at: source/Target/Target.cpp:3949
+                                                               idx);
+  assert(option_value);
+  return option_value->GetCurrentValue();
davide wrote:
> add an assertion message if you don't mind?
This code is mechanically duplicated from the other functions in this file, and I was actually planning on making another pass that auto-generates all of the properties and the accessor functions from a .inc file using an llvm-style HANDLE_foo() macro.


More information about the lldb-commits mailing list