`sycl::kernel_handler::get_specialization_constant` calls `__sycl_getComposite2020SpecConstantValue` on enumerated `value_type` about llvm HOT 4 OPEN

@victor-eds please, attach the code.

from llvm.

@victor-eds please, attach the code.

Added inline

from llvm.

Some massaging of our enable_ifs is required here:

We have a simple switch between compound and fundamental types, but we should probably use less C++-y terms like composite and scalar to treat more data types as scalar ones, even though they are compound per C++

from llvm.

Some massaging of our enable_ifs is required here:

llvm/sycl/include/sycl/kernel_handler.hpp

Lines 66 to 85 in 5fae0aa

	#ifdef __SYCL_DEVICE_ONLY__
	template <
	auto &S,
	typename T = typename std::remove_reference_t<decltype(S)>::value_type,
	std::enable_if_t<std::is_fundamental_v<T>> * = nullptr>
	__SYCL_ALWAYS_INLINE T getSpecializationConstantOnDevice() {
	const char *SymbolicID = __builtin_sycl_unique_stable_id(S);
	return __sycl_getScalar2020SpecConstantValue<T>(
	SymbolicID, &S, MSpecializationConstantsBuffer);
	}
	template <
	auto &S,
	typename T = typename std::remove_reference_t<decltype(S)>::value_type,
	std::enable_if_t<std::is_compound_v<T>> * = nullptr>
	__SYCL_ALWAYS_INLINE T getSpecializationConstantOnDevice() {
	const char *SymbolicID = __builtin_sycl_unique_stable_id(S);
	return __sycl_getComposite2020SpecConstantValue<T>(
	SymbolicID, &S, MSpecializationConstantsBuffer);
	}
	#endif // __SYCL_DEVICE_ONLY__

We have a simple switch between compound and fundamental types, but we should probably use less C++-y terms like composite and scalar to treat more data types as scalar ones, even though they are compound per C++

According to cppreference, enums are scalar types. Maybe using std::is_scalar would fix this?

from llvm.