[SYCL][Opaque pointer] Enable INTEL_SYCL_OPAQUEPOINTER_READY

[jsji]

This is to enable -DINTEL_SYCL_OPAQUEPOINTER_READY=1 by default,
so that we can start to remove the old code already removed by community (those guarded within #ifndef INTEL_SYCL_OPAQUEPOINTER_READY ).

• Set opauqe pointer ready
• Add flag to tablegen
• Remove opaque pointer from merge-extension-ivars.m
• Port barrier_intrinsic.ll to opaque pointer
• Remove intel-alloca-bitcast.ll as it is testing typed pointer path
• Revert changes to InferAddressSpaces/AMDGPU/icmp.ll
• Use private repo for testing purpose

[jsji]

This PR depends on a change in https://github.com/intel/vc-intrinsics and #10779 .
I'll update the corresponding commits (aa4b3af ) once they are merged.

But I'd like to kick off the review first as this is a big PR that changes multiple components.

Reviewers, please have a look. The plan is to enable this in sycl branch first, then flow it down to downstream branches,
then come back to sycl branch and remove the code within #ifndef INTEL_SYCL_OPAQUEPOINTER_READY and flow it down again.

[premanandrao]

I looked at the clang/test changes and they look okay to me.

[jsji]

Ping @intel/dpcpp-clang-driver-reviewers @intel/dpcpp-esimd-reviewers @intel/dpcpp-spirv-reviewers @intel/dpcpp-tools-reviewers @intel/llvm-reviewers-runtime .

Most changes in these components are just test update, please have a look. Thanks.
The plan is to merge this before EOD tomorrow. Thanks.

[sarnex]

esimd lgtm

[asudarsa]

Test changes in DPC++ Tools area looks good to me.
I do have a request for quick clarification for llvm-spirv changes here. Will it be possible to not make the changes directly here? We should be getting these changes as part of next pulldown from the Khronos SPIRV-LLVM-Translator repo. Adding @MrSidims and @jcranmer-intel for comment here.

Thanks a lot for putting up this major PR.

[jsji]

Test changes in DPC++ Tools area looks good to me. I do have a request for quick clarification for llvm-spirv changes here. Will it be possible to not make the changes directly here? We should be getting these changes as part of next pulldown from the Khronos SPIRV-LLVM-Translator repo. Adding @MrSidims and @jcranmer-intel for comment here.

Thanks a lot for putting up this major PR.

Thanks for the review. The SPIRV changes are here because these are local to sycl branch, we have removed the opaque pointer options in KhronosGroup/SPIRV-LLVM-Translator repo, but we added (kept) these options due to migration, this is to remove them so that we can in sync with KhronosGroup/SPIRV-LLVM-Translator repo again.

[jsji]

CI failure in SYCL-Unit :: Extensions/./ExtensionsTests/CommandGraphTest/GetProfilingInfoExceptionCheck is a bug in sycl branch will be fixed in #10954

[jsji]

@asudarsa @MrSidims @jcranmer-intel @intel/dpcpp-spirv-reviewers Can you have a look? Hopefully we can get this merged today. :)

[asudarsa]

SPIR-V changes are in sync with open source. So we should be ok.

Thanks

[jsji]

Thanks @asudarsa ! @intel/llvm-gatekeepers Can we get this merged? Thank you!

[jsji]

@intel/llvm-gatekeepers @bader OK to merge? Thank you!