[OpenCL] Allow optional __generic in __remove_address_space utility

[Topotuna]

Summary

Clang builtin utility __remove_address_space now works if generic
address space is not supported in C++ for OpenCL 2021.

[Anastasia]

LGTM! Thanks

[carlosgalvezp]

Hi! I believe this commit broke a number of CI jobs, would you be able to look into it?

https://lab.llvm.org/buildbot/#/builders/52/builds/12263

[Topotuna]

In D110155#3117786, @carlosgalvezp wrote:

Hi! I believe this commit broke a number of CI jobs, would you be able to look into it?

https://lab.llvm.org/buildbot/#/builders/52/builds/12263

Hi, I am aware it broke some of the builds. However, I cannot recreate the issue locally. Planning to revert the change shortly to investigate it further

[Topotuna]

Run line -triple is now set explicitly. Test moved from CodeGenOpenCLCXX directory to SemaOpenCLCXX

[Topotuna]

Run line argument reordering

[Anastasia]

LGTM! Thanks

[Topotuna]

Closed by commit: rGb5fef6dbfd6b4c84d61155fd2221064bcc1ecdcf

clang/lib/Headers/opencl-c-base.h

 #define as_size_t(x) __builtin_astype((x), size_t)
 #define as_ptrdiff_t(x) __builtin_astype((x), ptrdiff_t)
 #define as_intptr_t(x) __builtin_astype((x), intptr_t)
 #define as_uintptr_t(x) __builtin_astype((x), uintptr_t)
 
 // C++ for OpenCL - __remove_address_space
 #if defined(__OPENCL_CPP_VERSION__)
 template <typename _Tp> struct __remove_address_space { using type = _Tp; };
+#if defined(__opencl_c_generic_address_space)
 template <typename _Tp> struct __remove_address_space<__generic _Tp> {
   using type = _Tp;
 };
+#endif
 template <typename _Tp> struct __remove_address_space<__global _Tp> {
   using type = _Tp;
 };
 template <typename _Tp> struct __remove_address_space<__private _Tp> {
   using type = _Tp;
 };
 template <typename _Tp> struct __remove_address_space<__local _Tp> {
   using type = _Tp;

 
 #define kernel_exec(X, typen) __kernel \
   __attribute__((work_group_size_hint(X, 1, 1))) \
   __attribute__((vec_type_hint(typen)))
 
 #if defined(__OPENCL_CPP_VERSION__) || (__OPENCL_C_VERSION__ >= CL_VERSION_1_2)
 // OpenCL v1.2 s6.12.13, v2.0 s6.13.13 - printf
 
 int printf(__constant const char* st, ...) __attribute__((format(printf, 1, 2)));
 #endif
 
 #ifdef cl_intel_device_side_avc_motion_estimation
 
 #define CLK_AVC_ME_MAJOR_16x16_INTEL 0x0
 #define CLK_AVC_ME_MAJOR_16x8_INTEL 0x1
 #define CLK_AVC_ME_MAJOR_8x16_INTEL 0x2
 #define CLK_AVC_ME_MAJOR_8x8_INTEL 0x3

clang/test/CodeGenOpenCLCXX/remove-address-space.clcpp

-// RUN: %clang_cc1 %s -cl-std=clc++ -fdeclare-opencl-builtins -finclude-default-header
+// RUN: %clang_cc1 %s -cl-std=clc++1.0 -fdeclare-opencl-builtins -finclude-default-header
+// RUN: %clang_cc1 %s -cl-std=clc++2021 -fdeclare-opencl-builtins -finclude-default-header
+// RUN: %clang_cc1 %s -cl-std=clc++2021 -cl-ext=-__opencl_c_generic_address_space,-__opencl_c_pipes -fdeclare-opencl-builtins -finclude-default-header
 
 template<typename T, typename U>
 struct is_same {
   static const bool value = false;
 };
 
 template<typename T>
 struct is_same<T, T> {
   static const bool value = true;
 };
 
 void test_is_same() {
   static_assert(is_same<int, int>::value);
   static_assert(!is_same<int, float>::value);
   static_assert(!is_same<__private int, int>::value);
 }
 
 void test_remove_address_space() {
   static_assert(is_same<__remove_address_space<int>::type, int>::value,
                 "type without an address space unexpectedly modified by __remove_address_space");
+#if defined(__opencl_c_generic_address_space)
   static_assert(is_same<__remove_address_space<__generic int>::type, int>::value,
                 "__generic address space not removed by __remove_address_space");
+#endif
   static_assert(is_same<__remove_address_space<__global char>::type, char>::value,
                 "__global address space not removed by __remove_address_space");
   static_assert(is_same<__remove_address_space<__private ulong>::type, ulong>::value,
                 "__private address space not removed by __remove_address_space");
   static_assert(is_same<__remove_address_space<__local short>::type, short>::value,
                 "__local address space not removed by __remove_address_space");
   static_assert(is_same<__remove_address_space<__constant int3>::type, int3>::value,
                 "__constant address space not removed by __remove_address_space");
   static_assert(is_same<__remove_address_space<const volatile __global int>::type, const volatile int>::value,
                 "non-address-space qualifiers inappropriately removed by __remove_address_space");
 }