diff options
| author | Chris Forbes <chrisforbes@google.com> | 2015-11-25 08:32:46 +1300 |
|---|---|---|
| committer | Mark Lobodzinski <mark@lunarg.com> | 2015-11-26 13:41:26 -0700 |
| commit | dd8052e42b7f274af26cf80d8d375bde923b44df (patch) | |
| tree | 552c35546c9862533d76f131d45f2c17cf4d5d3c /layers/shader_checker.cpp | |
| parent | ae652a469dcae827d1317a65b1f51aa3c6ba6f50 (diff) | |
| download | usermoji-dd8052e42b7f274af26cf80d8d375bde923b44df.tar.xz | |
layers/SC: Delete disabled FS broadcast checking code
The support for collecting builtins on an interface is still kept
however -- we're likely to want to validate other builtin usage against
the pipeline config.
Diffstat (limited to 'layers/shader_checker.cpp')
| -rw-r--r-- | layers/shader_checker.cpp | 26 |
1 files changed, 0 insertions, 26 deletions
diff --git a/layers/shader_checker.cpp b/layers/shader_checker.cpp index 2f26cdc1..2697064c 100644 --- a/layers/shader_checker.cpp +++ b/layers/shader_checker.cpp @@ -899,32 +899,6 @@ validate_fs_outputs_against_render_pass(layer_data *my_data, VkDevice dev, shade collect_interface_by_location(my_data, dev, fs, spv::StorageClassOutput, outputs, builtin_outputs); - /* Check for legacy gl_FragColor broadcast: In this case, we should have no user-defined outputs, - * and all color attachment should be UNORM/SNORM/FLOAT. - */ -#if 0 - if (builtin_outputs.find(spv::BuiltInFragColor) != builtin_outputs.end()) { - if (outputs.size()) { - if (log_msg(my_data->report_data, VK_DBG_REPORT_ERROR_BIT, VK_OBJECT_TYPE_DEVICE, /*dev*/0, 0, SHADER_CHECKER_FS_MIXED_BROADCAST, "SC", - "Should not have user-defined FS outputs when using broadcast")) { - pass = false; - } - } - - for (unsigned i = 0; i < color_formats.size(); i++) { - unsigned attachmentType = get_format_type(color_formats[i]); - if (attachmentType == FORMAT_TYPE_SINT || attachmentType == FORMAT_TYPE_UINT) { - if (log_msg(my_data->report_data, VK_DBG_REPORT_ERROR_BIT, VK_OBJECT_TYPE_DEVICE, /*dev*/0, 0, SHADER_CHECKER_INTERFACE_TYPE_MISMATCH, "SC", - "CB format should not be SINT or UINT when using broadcast")) { - pass = false; - } - } - } - - return pass; - } -#endif - auto it = outputs.begin(); uint32_t attachment = 0; |
