[Lldb-commits] [PATCH] D43099: Make LLDB's clang module cache path customizable
Davide Italiano via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Feb 9 10:05:46 PST 2018
davide added inline comments.
Comment at: source/Target/Target.cpp:3949
+ 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.
Sure, even better :) It would be awesome if lldb didn't do its own command line handling but would use something like `cl::opt`, but I guess that's a battle for another day.
More information about the lldb-commits