[Lldb-commits] [PATCH] D11102: Set the default language to use when evaluating to that of the frame's CU.
Dawn Perchik via lldb-commits
lldb-commits at lists.llvm.org
Tue Sep 1 11:10:32 PDT 2015
dawn added a comment.
Just an update to let you know I'm still working on this (and a hundred other things)...
Comment at: source/Expression/ClangExpressionParser.cpp:227
@@ +226,3 @@
+ // for C++, and then we'll truly be able to follow the frame language. For now,
+ // we do "Want C -> get ObjC++", "Want ObjC -> get ObjC++" etc... But again,
+ // that is not a fundamental choice, it is an implementation necessity.
In the process of rewriting this patch, I realized I hadn't handled the "Want C -> get ObjC++" case, which, if the switch statement case for C is fixed to handle all the C variants, turns out to be needed. When that is fixed, we get a few new failures, including the test for the expr --language option. I'm trying to figure out how to rework those tests, but with everything enabling ObjC++ now, that's not so easy.
More information about the lldb-commits