[Lldb-commits] [PATCH] D11482: Add target setting to have the language follow the frame's CU.

Dawn Perchik 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.

See also:
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 mailing list