[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
Mon Nov 26 05:38:08 PST 2018


aleksandr.urakov added a reviewer: labath.
aleksandr.urakov marked an inline comment as done.
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;
----------------
labath wrote:
> aleksandr.urakov wrote:
> > 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?
> Instead of `ifdef` it would be better to do a `target->GetArchitecture().getTriple().getVendor() == llvm::Triple::Apple` and then `@skipIfDarwin` in the test. But it would be certainly better to remove this altogether :)
Yes, your variant is better! I'll do it in the way you have described if there will be objections against removing it at all.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54843/new/

https://reviews.llvm.org/D54843





More information about the lldb-commits mailing list