[SYCL][Matrix][E2E] Mark AMD/NVIDIA build targets as unsupported for some Matrix tests

[ayylol]

Adds UNSUPPORTED: target-nvidia, target-amd to matrix tests that also require aspect-ext_intel_matrix since these tests are not yet supported for those triples.

[dkhaldi]

since these tests do not build for any triples other than spir64

The tests can also build for specific target values like intel_gpu_pvc

[ayylol]

since these tests do not build for any triples other than spir64

The tests can also build for specific target values like intel_gpu_pvc

Would it be preferable to mark these instead with UNSUPPORTED: target-amd, target-nvidia, target-native_cpu? Those are the triples that i tested that didn't build.

Also curious, how are we currently testing for the other specific target values that you mention?

[ayylol]

@intel/sycl-matrix-reviewers @dkhaldi Hey, just wanted to continue this conversation regarding this pr, given that #16813 is dependent on the changes here.

[dkhaldi]

since these tests do not build for any triples other than spir64

The tests can also build for specific target values like intel_gpu_pvc

Would it be preferable to mark these instead with UNSUPPORTED: target-amd, target-nvidia, target-native_cpu? Those are the triples that i tested that didn't build.

I think that will be better as it really captures the real situation.
Just to double check, target-native_cpu means bypassing SPIRV right?

Also curious, how are we currently testing for the other specific target values that you mention?

We are not testing today for the specific target values as we don't specify any AOT. We only do jit compilation for these tests.

[ayylol]

I think that will be better as it really captures the real situation.

Ok, I've done that. There was one test that was failing to build for nvptx, which required gpu-intel-gen12 so I left the requirement as target-spir for it.

Just to double check, target-native_cpu means bypassing SPIRV right?

From what I know it is a different triple, so yes it wouldnt be spir. Though I dont currently have plans to enable that for the split build/run in CI so I omitted it.

[dkhaldi]

LGTM