[PATCH] D28349: [Frontend] The macro that describes the Objective-C bool type should be defined in C as well

Alex Lorenz via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 6 02:43:06 PST 2017


arphaman added a comment.

In https://reviews.llvm.org/D28349#636968, @doug.gregor wrote:

> The "!LangOpts.CPlusPlus" doesn't make sense to me. You're presumably defining this macro in C because the Objective-C runtime is usable from C, but that same logic applies to C++ code. It seems like we should be defining this macro unconditionally.


That's true. I was initially reluctant to define it for C++ as well since this macro is available on all platforms, but I guess there's no way around this.


Repository:
  rL LLVM

https://reviews.llvm.org/D28349





More information about the cfe-commits mailing list