[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 mailing list