clayborg added a comment. It would be much easier to read this patch if there were no renames from "*offset" to "*file_offset". Can we remove these extra renames where it isn't needed?
================ Comment at: source/Plugins/SymbolFile/DWARF/DWARFDebugInfo.h:15 #include <vector> +#include <mutex> ---------------- Why is this added here? Doesn't seem to be needed, or it should be moved to .cpp file? https://reviews.llvm.org/D40467 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits