thakis added a comment.
This is not NFC, is it? Before, this got evaluated as
`(!S.getLangOpts().OpenCLCPlusPlus && S.getLangOpts().OpenCLVersion < 200) ||
(S.getLangOpts().OpenCLVersion == 300 &&
!S.getOpenCLOptions().isSupported("__opencl_c_read_write_images",
S.getLangOpts()))`. Now, it's evaluated as `!S.getLangOpts().OpenCLCPlusPlus &&
(...rest...)` -- that is, `!S.getLangOpts().OpenCLCPlusPlus` used to be anded
with just one term and now it's anded with the whole thing.
Maybe it's worth to introduce some bool variables to make this less confusing
instead of this one very long term?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105892/new/
https://reviews.llvm.org/D105892
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
