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.


Repository:
  rL LLVM

http://reviews.llvm.org/D11482




_______________________________________________
lldb-commits mailing list
lldb-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits

Reply via email to