[Lldb-commits] [PATCH] D11482: Add target setting to have the language follow the frame's CU.
dawn+llvm at burble.org
Mon Jul 27 11:09:52 PDT 2015
dawn requested a review of this revision.
dawn added a comment.
I need guidance due to conflicting directions from Greg and Sean. Sean wants to be able to evaluate ObjC++ by default always, hence this feature was made optional. Greg feels the frame can be used for the language without an option, but that will break the ability to evaluate ObjC++ anytime.
http://reviews.llvm.org/D11102 - eval in language of frame's CU and add C++ language option to ObjC so tests will still work.
http://reviews.llvm.org/D11173 - rewrote test which tested ObjC types in C++ test case to be ObjC++ so test would still work.
We can either go with this patch, or http://reviews.llvm.org/D11102 and http://reviews.llvm.org/D11173. I would choose this patch because it makes the behavior optional, and everyone is happy.
Please advise. Thank you.
More information about the lldb-commits