aleksandr.urakov added inline comments.

================
Comment at: 
source/Plugins/ExpressionParser/Clang/ClangExpressionParser.cpp:401-404
-    // FIXME: the following language option is a temporary workaround,
-    // to "ask for C++, get ObjC++".  Apple hopes to remove this requirement on
-    // non-Apple platforms, but for now it is needed.
-    m_compiler->getLangOpts().ObjC = true;
----------------
Is it still necessary? We can do here something like `#ifdef __APPLE__`, but 
then the test will fail on Apple platforms. Can we somehow specify a platform 
requirement in the test?


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D54843



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to