[llvm] [DXIL] Add support for root signature flag element in DXContainer (PR #123147)
Chris B via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 12 13:19:17 PST 2025
================
@@ -280,11 +278,9 @@ Error DirectX::RootSignature::parse(StringRef Data) {
support::endian::read<uint32_t, llvm::endianness::little>(Current);
Current += sizeof(uint32_t);
- Expected<uint32_t> MaybeFlag =
- dxbc::RootSignatureValidations::validateRootFlag(FValue);
- if (Error E = MaybeFlag.takeError())
- return E;
- Flags = MaybeFlag.get();
+ if (dxbc::RootSignatureValidations::validateRootFlag(FValue))
+ return make_error<GenericBinaryError>("Invalid Root Signature flag");
----------------
llvm-beanz wrote:
Same feedback as above, we can make this a useful error.
https://github.com/llvm/llvm-project/pull/123147
More information about the llvm-commits
mailing list