Fix a few compile warnings across Vulkan tests
authorPyry Haulos <phaulos@google.com>
Mon, 9 Nov 2015 22:45:12 +0000 (14:45 -0800)
committerPyry Haulos <phaulos@google.com>
Tue, 10 Nov 2015 17:32:21 +0000 (09:32 -0800)
Change-Id: Ib58bf0f33e89efc449a40279656532767187d324

external/vulkancts/modules/vulkan/pipeline/vktPipelineReferenceRenderer.cpp
external/vulkancts/modules/vulkan/pipeline/vktPipelineStencilTests.cpp
external/vulkancts/modules/vulkan/vktRenderPassTests.cpp

index b9ebe7d..744ad67 100644 (file)
@@ -91,10 +91,10 @@ rr::BlendEquation mapVkBlendOp (VkBlendOp blendOp)
 
 tcu::BVec4 mapVkChannelFlags (VkChannelFlags flags)
 {
-       return tcu::BVec4(flags & VK_CHANNEL_R_BIT,
-                                         flags & VK_CHANNEL_G_BIT,
-                                         flags & VK_CHANNEL_B_BIT,
-                                         flags & VK_CHANNEL_A_BIT);
+       return tcu::BVec4((flags & VK_CHANNEL_R_BIT) != 0,
+                                         (flags & VK_CHANNEL_G_BIT) != 0,
+                                         (flags & VK_CHANNEL_B_BIT) != 0,
+                                         (flags & VK_CHANNEL_A_BIT) != 0);
 }
 
 rr::TestFunc mapVkCompareOp (VkCompareOp compareFunc)
index a527ecf..6346300 100644 (file)
@@ -72,7 +72,7 @@ bool isSupportedDepthStencilFormat (const InstanceInterface& instanceInterface,
 
        VK_CHECK(instanceInterface.getPhysicalDeviceFormatProperties(device, format, &formatProps));
 
-       return formatProps.optimalTilingFeatures & VK_FORMAT_FEATURE_DEPTH_STENCIL_ATTACHMENT_BIT;
+       return (formatProps.optimalTilingFeatures & VK_FORMAT_FEATURE_DEPTH_STENCIL_ATTACHMENT_BIT) != 0;
 }
 
 class StencilOpStateUniqueRandomIterator : public UniqueRandomIterator<VkStencilOpState>
index 6a0a3d0..3df609f 100644 (file)
@@ -1867,8 +1867,8 @@ void pushImageInitializationCommands (const DeviceInterface&                                                              vk,
                        };
                        const VkImageSubresourceRange range =
                        {
-                               hasDepthComponent(format.order) ? VK_IMAGE_ASPECT_DEPTH : 0
-                                       | hasStencilComponent(format.order) ? VK_IMAGE_ASPECT_STENCIL : 0,
+                               (VkImageAspectFlags)(hasDepthComponent(format.order) ? VK_IMAGE_ASPECT_DEPTH : 0
+                                                                        | hasStencilComponent(format.order) ? VK_IMAGE_ASPECT_STENCIL : 0),
                                0,
                                1,
                                0,
@@ -2632,19 +2632,19 @@ public:
        void    setColorStatus          (Status status)
        {
                DE_ASSERT(getColorStatus() == STATUS_UNDEFINED);
-               m_status |= ((deUint8)status) << COLOR_OFFSET;
+               m_status |= (deUint8)(status << COLOR_OFFSET);
        }
 
        void    setDepthStatus          (Status status)
        {
                DE_ASSERT(getDepthStatus() == STATUS_UNDEFINED);
-               m_status |= ((deUint8)status) << DEPTH_OFFSET;
+               m_status |= (deUint8)(status << DEPTH_OFFSET);
        }
 
        void    setStencilStatus        (Status status)
        {
                DE_ASSERT(getStencilStatus() == STATUS_UNDEFINED);
-               m_status |= ((deUint8)status) << STENCIL_OFFSET;
+               m_status |= (deUint8)(status << STENCIL_OFFSET);
        }
 
 private:
@@ -4166,7 +4166,7 @@ de::MovePtr<tcu::TestCaseGroup> createAttachmentAllocationTestGroup (tcu::TestCo
                                                                                                         VK_MEMORY_OUTPUT_COLOR_ATTACHMENT_BIT,
                                                                                                         VK_MEMORY_INPUT_COLOR_ATTACHMENT_BIT,
 
-                                                                                                        byRegion ? VK_TRUE : VK_FALSE));
+                                                                                                        byRegion ? (VkBool32)VK_TRUE : (VkBool32)VK_FALSE));
                                }
 
                                const RenderPass                                        renderPass              (attachments, subpasses, deps);