[llvm] [DXIL] Adding support to RootSignatureFlags in obj2yaml (PR #122396)
Chris B via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 6 20:06:18 PST 2025
================
@@ -228,6 +242,53 @@ void DXContainer::PartIterator::updateIteratorImpl(const uint32_t Offset) {
IteratorState.Offset = Offset;
}
+Error DirectX::RootSignature::parse(StringRef Data) {
+ const char *Current = Data.begin();
+
+ // Root Signature headers expects 6 integers to be present.
+ if (Data.size() < 6 * sizeof(uint32_t)) {
+ return parseFailed("Invalid data. Too small.");
----------------
llvm-beanz wrote:
The error here should probably be more descriptive. Imagine having a tool print this error? Without context it doesn't really tell you what went wrong.
Maybe something more like "Invalid root signature, insufficient space for header."
https://github.com/llvm/llvm-project/pull/122396
More information about the llvm-commits
mailing list