Skip to content

Commit 835bb55

Browse files
committed
Enable most of the cbuffer tests for clang
These have been working for some time but we never enabled the tests.
1 parent 9825eb1 commit 835bb55

File tree

5 files changed

+15
-3
lines changed

5 files changed

+15
-3
lines changed

test/Feature/CBuffer/arrays-16bit.test

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,8 +59,8 @@ DescriptorSets:
5959
# DXC's vulkan support does not layout cbuffers compatibly with DXIL
6060
# UNSUPPORTED: Vulkan
6161

62-
# https://github.com/llvm/llvm-project/issues/110722
63-
# XFAIL: Clang
62+
# https://github.com/llvm/llvm-project/issues/138996
63+
# UNSUPPORTED: Clang
6464

6565
# RUN: split-file %s %t
6666
# RUN: %dxc_target -enable-16bit-types -T cs_6_5 -Fo %t.o %t/source.hlsl

test/Feature/CBuffer/lit.local.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
if 'Clang' in config.available_features:
1+
if 'Clang-Vulkan' in config.available_features:
22
config.unsupported = True
33

44
# CBuffer bindings seem to be broken under metal

test/Feature/CBuffer/structs.test

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,10 @@ DescriptorSets:
9797
# DXC's vulkan support does not layout cbuffers compatibly with DXIL
9898
# UNSUPPORTED: Vulkan
9999

100+
+# Clang trips on 2-element vectors in structs:
101+
+# https://github.com/llvm/llvm-project/issues/123968
102+
+# UNSUPPORTED: Clang
103+
100104
# RUN: split-file %s %t
101105
# RUN: %dxc_target -T cs_6_5 -Fo %t.o %t/source.hlsl
102106
# RUN: %offloader %t/pipeline.yaml %t.o | FileCheck %s

test/Feature/CBuffer/vectors-64bit.test

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,10 @@ DescriptorSets:
6363
# DXC's vulkan support does not layout cbuffers compatibly with DXIL
6464
# UNSUPPORTED: Vulkan
6565

66+
# Clang trips on 3-element vectors in structs:
67+
# https://github.com/llvm/llvm-project/issues/123968
68+
# UNSUPPORTED: Clang
69+
6670
# RUN: split-file %s %t
6771
# RUN: %dxc_target -T cs_6_5 -Fo %t.o %t/source.hlsl
6872
# RUN: %offloader %t/pipeline.yaml %t.o | FileCheck %s

test/Feature/CBuffer/vectors.test

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,10 @@ DescriptorSets:
6363
# DXC's vulkan support does not layout cbuffers compatibly with DXIL
6464
# UNSUPPORTED: Vulkan
6565

66+
# Clang trips on 3-element vectors in structs:
67+
# https://github.com/llvm/llvm-project/issues/123968
68+
# UNSUPPORTED: Clang
69+
6670
# RUN: split-file %s %t
6771
# RUN: %dxc_target -T cs_6_5 -Fo %t.o %t/source.hlsl
6872
# RUN: %offloader %t/pipeline.yaml %t.o | FileCheck %s

0 commit comments

Comments
 (0)