Minor stylistic issues, otherwise LGTM! http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFContext.cpp File lib/DebugInfo/DWARFContext.cpp (right): http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFContext.cpp#n... lib/DebugInfo/DWARFContext.cpp:161: functionName = name; I'd prefer to have the variable declaration into the if if (const char *name = function_die->getSubprogramName(cu)) functionName = name; http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFDebugInfoEntr... File lib/DebugInfo/DWARFDebugInfoEntry.cpp (right): http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFDebugInfoEntr... lib/DebugInfo/DWARFDebugInfoEntry.cpp:467: return name; Here, folding the name variable into the if would eliminate the double parens and it's easier to grasp the code as the variable lifetime is limited. There are more instances of this below.
http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFContext.cpp File lib/DebugInfo/DWARFContext.cpp (right): http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFContext.cpp#n... lib/DebugInfo/DWARFContext.cpp:161: functionName = name; On 2012/07/17 15:13:17, bkramer wrote: > I'd prefer to have the variable declaration into the if > > if (const char *name = function_die->getSubprogramName(cu)) > functionName = name; Done. http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFDebugInfoEntr... File lib/DebugInfo/DWARFDebugInfoEntry.cpp (right): http://codereview.appspot.com/6409051/diff/1/lib/DebugInfo/DWARFDebugInfoEntr... lib/DebugInfo/DWARFDebugInfoEntry.cpp:467: return name; On 2012/07/17 15:13:17, bkramer wrote: > Here, folding the name variable into the if would eliminate the double parens > and it's easier to grasp the code as the variable lifetime is limited. There are > more instances of this below. Done.