[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