nodejs / llnode

An lldb plugin for Node.js and V8, which enables inspection of JavaScript states for insights into Node.js processes and their core dumps.
Other
1.15k stars 99 forks source link

src: improve messages on PrintInDebugMode #293

Closed mmarchini closed 4 years ago

mmarchini commented 4 years ago

Add function, file and line information to debug messages.

mmarchini commented 4 years ago

Example output:

[llnode][Failure ../src/error.cc:35] Failed to find symbol nodedbg_const_Environment__kContextEmbedderDataIndex__int                                                                                                                                                                                                                                                                                                                         
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_off_fp_marker                                                                                                                                                                                                                                                                                                                                                               
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_frametype_JavaScriptFrame                                                                                                                                                                                                                                                                                                                                                   
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant frametype_JavaScriptFrame, default to -1                                                                                                                                                                                                                                                                                                                              
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Context__closure_index__int                                                                                                                                                                                                                                                                                                                                           
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_context_idx_closure                                                                                                                                                                                                                                                                                                                                                         
[llnode][LoadConstant ../src/constants.cc:114] Failed to load constant class_Context__closure_index__int, fallback context_idx_closure, default to -1                                                                                                                                                                                                                                                                                        
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Context__previous_index__int                                                                                                                                                                                                                                                                                                                                          
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Context__native_index__int                                                                                                                                                                                                                                                                                                                                            
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Context__min_context_slots__int                                                                                                                                                                                                                                                                                                                                       
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Map__instance_attributes__int                                                                                                                                                                                                                                                                                                                                         
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Map__inobject_properties_or_constructor_function_index__int                                                                                                                                                                                                                                                                                                           
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Map__inobject_properties__int                                                                                                                                                                                                                                                                                                                                         
[llnode][LoadConstant ../src/constants.cc:114] Failed to load constant class_Map__inobject_properties_or_constructor_function_index__int, fallback class_Map__inobject_properties__int, default to -1                                                                                                                                                                                                                                        
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_Map__instance_size__int                                                                                                                                                                                                                                                                                                                                               
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_bit_field3_dictionary_map_shift                                                                                                                                                                                                                                                                                                                                             
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_type_UncompiledDataWithoutPreParsedScope__UNCOMPILED_DATA_WITHOUT_PRE_PARSED_SCOPE_TYPE                                                                                                                                                                                                                                                                                     
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant type_UncompiledDataWithoutPreParsedScope__UNCOMPILED_DATA_WITHOUT_PRE_PARSED_SCOPE_TYPE, default to -1                                                                                                                                                                                                                                                                
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_type_UncompiledDataWithPreParsedScope__UNCOMPILED_DATA_WITH_PRE_PARSED_SCOPE_TYPE                                                                                                                                                                                                                                                                                           
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant type_UncompiledDataWithPreParsedScope__UNCOMPILED_DATA_WITH_PRE_PARSED_SCOPE_TYPE, default to -1                                                                                                                                                                                                                                                                      
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__raw_name__Object                                                                                                                                                                                                                                                                                                                                  
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__name__Object                                                                                                                                                                                                                                                                                                                                      
[llnode][LoadConstant ../src/constants.cc:114] Failed to load constant class_SharedFunctionInfo__raw_name__Object, fallback class_SharedFunctionInfo__name__Object, default to -1                                                                                                                                                                                                                                                            
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__inferred_name__String                                                                                                                                                                                                                                                                                                                             
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__script_or_debug_info__Object                                                                                                                                                                                                                                                                                                                      
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_SharedFunctionInfo__script_or_debug_info__Object, default to -1                                                                                                                                                                                                                                                                                                 
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__scope_info__ScopeInfo                                                                                                                                                                                                                                                                                                                             
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_SharedFunctionInfo__scope_info__ScopeInfo, default to -1                                                                                                                                                                                                                                                                                                        
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_SharedFunctionInfo__compiler_hints__int                                                                                                                                                                                                                                                                                                                               
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_SharedFunctionInfo__compiler_hints__int, default to -1                                                                                                                                                                                                                                                                                                          
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_prop_type_mask                                                                                                                                                                                                                                                                                                                                                              
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant prop_type_mask, default to -1                                                                                                                                                                                                                                                                                                                                         
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_UncompiledData__inferred_name__String                                                                                                                                                                                                                                                                                                                                 
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_UncompiledData__inferred_name__String, default to -1                                                                                                                                                                                                                                                                                                            
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_UncompiledData__start_position__int32_t                                                                                                                                                                                                                                                                                                                               
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_UncompiledData__start_position__int32_t, default to -1                                                                                                                                                                                                                                                                                                          
[llnode][Failure ../src/error.cc:35] Failed to find symbol v8dbg_class_UncompiledData__end_position__int32_t                                                                                                                                                                                                                                                                                                                                 
[llnode][LoadConstant ../src/constants.cc:100] Failed to load constant class_UncompiledData__end_position__int32_t, default to -1                                                                                                                                                                                                                                                                                                            
[llnode][Stringify<llnode::v8::HeapObject> ../src/printer.cc:761] Unknown HeapObject Type 1090 at 0x0000022192016939                                                                                                                                                                                                                                                                                                                         
codecov-io commented 4 years ago

Codecov Report

Merging #293 into master will increase coverage by 0.55%. The diff coverage is 40%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #293      +/-   ##
==========================================
+ Coverage   79.48%   80.04%   +0.55%     
==========================================
  Files          33       33              
  Lines        4216     4219       +3     
==========================================
+ Hits         3351     3377      +26     
+ Misses        865      842      -23
Impacted Files Coverage Δ
src/error.h 85.71% <ø> (ø) :arrow_up:
src/llscan.cc 61.25% <0%> (-0.08%) :arrow_down:
src/llnode.cc 74.2% <0%> (ø) :arrow_up:
src/llv8.cc 76.21% <0%> (+2.23%) :arrow_up:
src/printer.cc 79.55% <100%> (+0.03%) :arrow_up:
src/error.cc 84.21% <100%> (ø) :arrow_up:
src/llv8-inl.h 93.64% <100%> (+1.05%) :arrow_up:
src/constants.cc 81.35% <66.66%> (ø) :arrow_up:
src/llv8-constants.cc 83.49% <0%> (+0.97%) :arrow_up:
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a501635...2dfee96. Read the comment docs.

mmarchini commented 4 years ago

Landed in 9449d99