inbelic wrote:
Closing as this is superseded by
https://github.com/llvm/llvm-project/pull/149608 and the documentation from
[here](https://github.com/llvm/wg-hlsl/blob/main/proposals/0002-root-signature-in-clang.md).
https://github.com/llvm/llvm-project/pull/88781
_
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/88781
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -24,6 +26,48 @@ using namespace llvm;
using namespace llvm::dxil;
namespace {
+static const char *ResourceClassToString(llvm::dxil::ResourceClass Class) {
+ switch (Class) {
+ case ResourceClass::SRV:
+return "SRV";
+ case ResourceClass::UAV:
+return "UAV";
+ ca
@@ -316,6 +316,32 @@ findOverlappingRanges(llvm::SmallVector &Infos)
{
return Overlaps;
}
+llvm::SmallVector
+findUnboundRanges(const llvm::SmallVectorImpl &Ranges,
+ const llvm::ArrayRef &Bindings) {
+ llvm::SmallVector Unbounds;
+ for (const auto &Range
@@ -84,8 +128,150 @@ static void reportOverlappingBinding(Module &M,
DXILResourceMap &DRM) {
}
}
+static void reportRegNotBound(Module &M,
+ llvm::hlsl::rootsig::RangeInfo Unbound) {
+ SmallString<128> Message;
+ raw_svector_ostream OS(Message
@@ -136,6 +136,51 @@ struct OverlappingRanges {
llvm::SmallVector
findOverlappingRanges(llvm::SmallVector &Infos);
+class RootSignatureBindingValidation {
+private:
+ llvm::SmallVector Bindings;
+ struct TypeRange {
+uint32_t Start;
+uint32_t End;
+ };
+ std::unord
@@ -316,6 +316,32 @@ findOverlappingRanges(llvm::SmallVector &Infos)
{
return Overlaps;
}
+llvm::SmallVector
+findUnboundRanges(const llvm::SmallVectorImpl &Ranges,
inbelic wrote:
```suggestion
findUnboundRanges(ArrayRef &Ranges,
```
https://github.com/ll
https://github.com/inbelic created
https://github.com/llvm/llvm-project/pull/148941
- it was noted,
[here](https://github.com/llvm/llvm-project/pull/145795#discussion_r2208118547),
that by accidently not specifying this explicitly as a float it will cause a
build warning on MSVC
- this commi
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/148649
>From ad3211d057b106bd713374369d567d0c9f08459f Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Mon, 14 Jul 2025 15:17:53 +
Subject: [PATCH 1/3] [NFC][SemaHLSL] Remove check dependent on
non-deterministic
inbelic wrote:
Although, thinking a bit more, we can actually remove this test entirely. The
FileCheck portion was just a nicety to check that the SourceLocation was
pointing correctly for this case. However, this is redundant with the other
checks that exist to do so.
https://github.com/llvm
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/148649
>From ad3211d057b106bd713374369d567d0c9f08459f Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Mon, 14 Jul 2025 15:17:53 +
Subject: [PATCH 1/2] [NFC][SemaHLSL] Remove check dependent on
non-deterministic
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/148649
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
inbelic wrote:
@guy-david @bevin-hansson This resolved the build issue for me locally when
using `DLLVM_ENABLE_EXSTENSIVE_CHECKS`, can you confirm this resolves the issue
for you as well?
https://github.com/llvm/llvm-project/pull/148649
___
cfe-comm
https://github.com/inbelic created
https://github.com/llvm/llvm-project/pull/148649
- when there are duplicate `RangeInfo`s created and we will attempt to
`llvm::sort` or `llvm::stable_sort` them, it does not appear deterministic in
which order they will be sorted (because they are equivalent)
inbelic wrote:
@guy-david Thanks for pointing this out, I have a similar report
[here](https://github.com/llvm/llvm-project/pull/147115#issuecomment-3069543491).
I am taking a look into both now, but I suspect they might be the same issue.
https://github.com/llvm/llvm-project/pull/147800
_
inbelic wrote:
Hey @bevin-hansson, thanks for pointing this out. Building locally and taking a
look into it now.
https://github.com/llvm/llvm-project/pull/147115
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/147832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147832
>From 02e7ad8a92e01b19d85f9bedf831aac161439ccb Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 21:21:53 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Implement multiple diagnostics in
`Root
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147832
>From 02e7ad8a92e01b19d85f9bedf831aac161439ccb Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 21:21:53 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Implement multiple diagnostics in
`Root
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/145795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145795
>From 43991791409d376e682c6a482941fcc4cd43b0bc Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 19:25:19 +
Subject: [PATCH 1/9] add basic integer range validations
---
.../clang/Basic/Diag
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147832
>From 02e7ad8a92e01b19d85f9bedf831aac161439ccb Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 21:21:53 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Implement multiple diagnostics in
`Root
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147832
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1083,6 +1084,90 @@ void SemaHLSL::ActOnFinishRootSignatureDecl(
bool SemaHLSL::handleRootSignatureElements(
ArrayRef Elements) {
+ // Define some common error handling functions
+ bool HadError = false;
+ auto ReportError = [this, &HadError](SourceLocation Loc, uint
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145795
>From 43991791409d376e682c6a482941fcc4cd43b0bc Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 19:25:19 +
Subject: [PATCH 1/8] add basic integer range validations
---
.../clang/Basic/Diag
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/147800
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147800
>From a510b930402cd4a6a00f253093d1a3ddfab918ac Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 9 Jul 2025 15:50:55 +
Subject: [PATCH 01/11] remove confusing hlsl_unexpected_end_of_params
diagnostic
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147800
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/147115
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147115
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -14,10 +14,129 @@
#ifndef LLVM_LIB_TARGET_DIRECTX_DXILPOSTOPTIMIZATIONVALIDATION_H
#define LLVM_LIB_TARGET_DIRECTX_DXILPOSTOPTIMIZATIONVALIDATION_H
+#include "DXILRootSignature.h"
+#include "llvm/ADT/IntervalMap.h"
+#include "llvm/Analysis/DXILResource.h"
#include "llvm/IR
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147115
>From 564f6995f40d80acddbda1fce58ddec38613d9fa Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 18:36:38 +
Subject: [PATCH 01/15] nfc: introduce wrapper `RootSignatureElement` around
`Root
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147115
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/147350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,13 @@
+// RUN: %clang_cc1 -triple dxil-pc-shadermodel6.3-library -x hlsl
-fsyntax-only %s -verify
+
+// expected-no-diagnostics
+
+// Test that we have consistent behaviour for comma parsing. Namely:
+// - a single trailing comma is allowed after any parameter
+// - a
inbelic wrote:
Added a test to show that it is consistent in allowing a trailing comma after
parameter/values
https://github.com/llvm/llvm-project/pull/147350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailm
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147350
>From b8644c7d9a44f9480cdbe0b3c46f0899cdcffc28 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Sat, 5 Jul 2025 01:41:26 +
Subject: [PATCH 1/9] fix for root elements
---
clang/lib/Parse/ParseHLSLRootSigna
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147350
>From b8644c7d9a44f9480cdbe0b3c46f0899cdcffc28 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Sat, 5 Jul 2025 01:41:26 +
Subject: [PATCH 1/7] fix for root elements
---
clang/lib/Parse/ParseHLSLRootSigna
@@ -34,3 +34,7 @@ void bad_root_signature_5() {}
// expected-error@+1 {{expected ')' to denote end of parameters, or, another
valid parameter of RootConstants}}
[RootSignature(MultiLineRootSignature)]
void bad_root_signature_6() {}
+
+// expected-error@+1 {{expected end of str
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/147117
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147117
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147350
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147084
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic ready_for_review
https://github.com/llvm/llvm-project/pull/147084
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147084
>From 34d3879022f65d51002d3abe3aec50a952ca6e18 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 4 Jul 2025 00:13:32 +
Subject: [PATCH 1/6] nfc: add phony ASTContext and StringLiteral to ParseHLSL
---
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/145828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/147084
>From abef31336baec05dafe2cc7ae32e753790322f68 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 4 Jul 2025 00:13:32 +
Subject: [PATCH 1/5] nfc: add phony ASTContext and StringLiteral to ParseHLSL
---
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/147084
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -22,3 +23,14 @@ void bad_root_signature_4() {}
// expected-error@+1 {{expected ')' to denote end of parameters, or, another
valid parameter of RootConstants}}
[RootSignature("RootConstants(b0, num32BitConstants = 1, invalid)")]
void bad_root_signature_5() {}
+
+#define Mult
https://github.com/inbelic created
https://github.com/llvm/llvm-project/pull/147084
The `SourceLocation` of a `RootSignatureToken` is incorrectly set to be the
"offset" into the concatenated string that denotes the rootsignature. This
causes an issue when the `StringLiteral` is a multi-line ex
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145828
>From e616ddee9cfaaaf697e4bd83d18e2d10ea86 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:39:13 +
Subject: [PATCH 1/6] update `setDefaultFlags`
---
.../clang/Parse/ParseHLSLRootS
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/146124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145828
>From e616ddee9cfaaaf697e4bd83d18e2d10ea86 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:39:13 +
Subject: [PATCH 1/6] update `setDefaultFlags`
---
.../clang/Parse/ParseHLSLRootS
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/146124
>From 91f0a282662bf62e35b6b6254cb96e48a2062162 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:56:24 +
Subject: [PATCH 1/7] nfc: move resource range analysis to `HLSLRootSignature`
---
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/146124
>From 91f0a282662bf62e35b6b6254cb96e48a2062162 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:56:24 +
Subject: [PATCH 1/6] nfc: move resource range analysis to `HLSLRootSignature`
---
@@ -0,0 +1,56 @@
+//===- RootSignatureMetadata.h - HLSL Root Signature helpers
--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -103,7 +51,7 @@ class ResourceRange {
getOverlapping(const RangeInfo &Info) const;
// Return the mapped RangeInfo at X or nullptr if no mapping exists
- LLVM_ABI const RangeInfo *lookup(uint32_t X) const;
+ const RangeInfo *lookup(uint32_t X) const;
i
@@ -0,0 +1,194 @@
+//===- RootSignatureMetadata.h - HLSL Root Signature helpers
--===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Ap
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145828
>From e616ddee9cfaaaf697e4bd83d18e2d10ea86 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:39:13 +
Subject: [PATCH 1/6] update `setDefaultFlags`
---
.../clang/Parse/ParseHLSLRootS
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/146124
>From 91f0a282662bf62e35b6b6254cb96e48a2062162 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:56:24 +
Subject: [PATCH 1/5] nfc: move resource range analysis to `HLSLRootSignature`
---
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/146124
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145828
>From e616ddee9cfaaaf697e4bd83d18e2d10ea86 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 27 Jun 2025 16:39:13 +
Subject: [PATCH 1/5] update `setDefaultFlags`
---
.../clang/Parse/ParseHLSLRootS
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/145986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -274,11 +286,23 @@ TEST(HLSLRootSignatureTest, NoneRootFlagsDump) {
}
TEST(HLSLRootSignatureTest, AllRootFlagsDump) {
- RootFlags Flags = RootFlags::ValidFlags;
+ using llvm::dxbc::RootFlags;
+ auto ValidRootFlags = RootFlags::AllowInputAssemblerInputLayout |
+
https://github.com/inbelic deleted
https://github.com/llvm/llvm-project/pull/145986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -274,11 +286,23 @@ TEST(HLSLRootSignatureTest, NoneRootFlagsDump) {
}
TEST(HLSLRootSignatureTest, AllRootFlagsDump) {
- RootFlags Flags = RootFlags::ValidFlags;
+ using llvm::dxbc::RootFlags;
+ auto ValidRootFlags = RootFlags::AllowInputAssemblerInputLayout |
+
https://github.com/inbelic approved this pull request.
https://github.com/llvm/llvm-project/pull/146448
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -200,38 +81,38 @@ struct DescriptorTableClause {
uint32_t NumDescriptors = 1;
uint32_t Space = 0;
uint32_t Offset = DescriptorTableOffsetAppend;
- DescriptorRangeFlags Flags;
+ dxbc::DescriptorRangeFlags Flags;
void setDefaultFlags() {
switch (Type) {
@@ -205,20 +205,22 @@ TEST_F(ParseHLSLRootSignatureTest,
ValidParseDTClausesTest) {
ASSERT_EQ(std::get(Elem).Space, 0u);
ASSERT_EQ(std::get(Elem).Offset,
DescriptorTableOffsetAppend);
+ auto ValidDescriptorRangeFlags = llvm::dxbc::DescriptorRangeFlags(0x1000f)
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -404,7 +417,7 @@ TEST_F(ParseHLSLRootSignatureTest, ValidSamplerFlagsTest) {
ASSERT_TRUE(std::holds_alternative(Elem));
ASSERT_EQ(std::get(Elem).Type, ClauseType::Sampler);
ASSERT_EQ(std::get(Elem).Flags,
-DescriptorRangeFlags::ValidSamplerFlags);
-
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
inbelic wrote:
This pr have been converted to a draft until
https://github.com/llvm/llvm-project/issues/145940 and
https://github.com/llvm/llvm-project/issues/145815 are resolved and the logic
can be re-used.
https://github.com/llvm/llvm-project/pull/145795
___
https://github.com/inbelic converted_to_draft
https://github.com/llvm/llvm-project/pull/145795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/145827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145827
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -18,3 +18,7 @@ void bad_root_signature_3() {}
[RootSignature("DescriptorTable(), invalid")] // expected-error {{expected end
of stream to denote end of parameters, or, another valid parameter of
RootSignature}}
void bad_root_signature_4() {}
+
+// expected-error@+1 {{expe
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145828
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic created
https://github.com/llvm/llvm-project/pull/145828
This pr updates `setDefaultFlags` in `HLSLRootSignature.h` to account for which
version it should initialize the default flag values for.
- Updates `setDefaultFlags` with a `Version` argument and initializes th
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145795
>From 7e3f1398ae2ba147822f96dd262a2c6127f0ff68 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 25 Jun 2025 18:16:12 +
Subject: [PATCH 1/8] simple prototype first error
---
.../clang/Basic/Diagnostic
https://github.com/inbelic edited
https://github.com/llvm/llvm-project/pull/145795
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/144813
>From 6af81bb12430c28f22eba073a38abd7875a2e348 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 18 Jun 2025 21:47:31 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Add option to specify version of
RootS
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/144813
>From 6af81bb12430c28f22eba073a38abd7875a2e348 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 18 Jun 2025 21:47:31 +
Subject: [PATCH 1/5] [HLSL][RootSignature] Add option to specify version of
RootS
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/144813
>From 6af81bb12430c28f22eba073a38abd7875a2e348 Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Wed, 18 Jun 2025 21:47:31 +
Subject: [PATCH 1/7] [HLSL][RootSignature] Add option to specify version of
RootS
https://github.com/inbelic closed
https://github.com/llvm/llvm-project/pull/145109
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/144769
>From 3761e5204d2d435cc09f392fd2e53c2125fac0ed Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 6 Jun 2025 20:46:10 +
Subject: [PATCH 1/3] [HLSL][RootSignature] Plug-in serialization and add full
samp
@@ -1116,6 +1116,52 @@ bool
SemaHLSL::handleRootSignatureDecl(HLSLRootSignatureDecl *D,
Info.Space = Descriptor->Space;
Info.Visibility = Descriptor->Visibility;
Infos.push_back(Info);
+} else if (const auto *Constants =
+ std::get_if(&E
https://github.com/inbelic deleted
https://github.com/llvm/llvm-project/pull/144813
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/144769
>From 3761e5204d2d435cc09f392fd2e53c2125fac0ed Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 6 Jun 2025 20:46:10 +
Subject: [PATCH 1/4] [HLSL][RootSignature] Plug-in serialization and add full
samp
https://github.com/inbelic updated
https://github.com/llvm/llvm-project/pull/145109
>From 447dca922263f5a4abec27d8e32302dc1b7a1b5b Mon Sep 17 00:00:00 2001
From: Finn Plummer
Date: Fri, 20 Jun 2025 18:45:55 +
Subject: [PATCH 1/8] add collection of RootConstants
---
clang/lib/Sema/SemaHLSL
1 - 100 of 688 matches
Mail list logo
