[Lldb-commits] [PATCH] D54843: [Expr] Check the language before ignoring Objective C keywords

Aleksandr Urakov via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Nov 23 01:50:45 PST 2018


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





More information about the lldb-commits mailing list