[Lldb-commits] [PATCH] D53759: [PDB] Support PDB-backed expressions evaluation
Stella Stamenova via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Oct 26 10:21:34 PDT 2018
stella.stamenova accepted this revision.
stella.stamenova added a comment.
This revision is now accepted and ready to land.
A couple of comments, but looks good otherwise. I'd wait for someone else to have a look as well.
================
Comment at: source/Plugins/Process/Windows/Common/ProcessWindows.cpp:959
+ // It may be an expression evaluation crash.
+ SetPrivateState(eStateStopped);
}
----------------
Are we going to determine later whether it is supposed to be a crash or just never crash on second chance exceptions?
================
Comment at: source/Plugins/SymbolFile/PDB/PDBASTParser.cpp:1085
+ } else
+ set = &m_namespaces;
+ assert(set);
----------------
Nit: Maybe add curly braces here. Since the if statement has them, the code is easier to read if the else does as well.
Repository:
rLLDB LLDB
https://reviews.llvm.org/D53759
More information about the lldb-commits
mailing list