================ @@ -6169,9 +6169,13 @@ class HLSLAttributedResourceType : public Type, public llvm::FoldingSetNode { // Data gathered from HLSL resource attributes llvm::dxil::ResourceClass ResourceClass; uint8_t IsROV : 1; - Attributes(llvm::dxil::ResourceClass ResourceClass, bool IsROV) - : ResourceClass(ResourceClass), IsROV(IsROV) {} - Attributes() : ResourceClass(llvm::dxil::ResourceClass::UAV), IsROV(0) {} + uint8_t RowAccess : 1; + Attributes(llvm::dxil::ResourceClass ResourceClass, bool IsROV, + bool RowAccess) + : ResourceClass(ResourceClass), IsROV(IsROV), RowAccess(RowAccess) {} + Attributes() + : ResourceClass(llvm::dxil::ResourceClass::UAV), IsROV(0), + RowAccess(0) {} ---------------- bogner wrote:
Is it clearer to spell these `false` rather than `0`? https://github.com/llvm/llvm-project/pull/107954 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits