jingham added a comment. Other than the request for a more logging, this seems fine.
================ Comment at: source/Plugins/ExpressionParser/Clang/IRForTarget.cpp:514-515 + default: + encoding_flags = 0x0600; /* fall back to 0x0600, kCFStringEncodingASCII */ + } + Constant *encoding_arg = ConstantInt::get(i32_ty, encoding_flags, false); ---------------- The default case here expresses "I don't know what this string is so I'm just going to try ASCII". That's fine, but we should log the unknown kind so that when this happens in the wild and we can't get a repro case, we'll be able to figure out something from the log. Repository: rL LLVM https://reviews.llvm.org/D27291 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
