aboutsummaryrefslogtreecommitdiff
path: root/layers/shader_checker.cpp
diff options
context:
space:
mode:
authorChris Forbes <chrisforbes@google.com>2015-11-25 08:32:46 +1300
committerMark Lobodzinski <mark@lunarg.com>2015-11-26 13:41:26 -0700
commitdd8052e42b7f274af26cf80d8d375bde923b44df (patch)
tree552c35546c9862533d76f131d45f2c17cf4d5d3c /layers/shader_checker.cpp
parentae652a469dcae827d1317a65b1f51aa3c6ba6f50 (diff)
downloadusermoji-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.cpp26
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;