apolyakov added inline comments.
================
Comment at: tools/lldb-mi/MICmdCmdSymbol.cpp:24
+namespace {
+inline const CMICmnMIValueTuple
+CreateMITuplePCLine(const uint32_t addr, const uint32_t line_number) {
----------------
aprantl wrote:
> Please remove the `inline` keyword. LLVM will inline if it's profitable
> anyway.
> Also: could this be a constructor of CMImnMIValueTuple?
I think it could not. There are specific strings like "0x", "pc", "line" in
this function which don't have any relation to CMICmnMIValueTuple class. Also,
passing them as arguments doesn't seem like a good way. But it can be
discussed, maybe someone will suggest a good approach for dealing with it.
https://reviews.llvm.org/D48802
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
