Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land.
LGTM! Thanks. I have made a minor suggestion to testing that could be addressed in the final commit. ================ Comment at: clang/test/Misc/nvptx.unsupported_core.cl:1 +// RUN: %clang_cc1 -cl-std=CL2.0 -triple nvptx-unknown-unknown -Wpedantic-core-features %s 2> %t +// RUN: FileCheck < %t %s ---------------- We could add a line with `-cl-std=CLC++` to check the full wording of the new diagnostic. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101087/new/ https://reviews.llvm.org/D101087 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits