From 3d112a7cd49656da3dde5f9b476b6ddc2d984528 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Wed, 18 Oct 2017 11:22:43 -0700 Subject: i965: Move fs_inst::has_side_effects()'s eot check to the parent class. This eliminates a layer of wrapping, and makes a backend_instruction sufficient. The downside is that it exposes 'eot' to the vec4 backend, which it doesn't need, but can basically happily ignore. Reviewed-by: Matt Turner Tested-by: Pallavi G --- src/intel/compiler/brw_fs.cpp | 6 ------ 1 file changed, 6 deletions(-) (limited to 'src/intel/compiler/brw_fs.cpp') diff --git a/src/intel/compiler/brw_fs.cpp b/src/intel/compiler/brw_fs.cpp index 66cb33131b6..30e8841242d 100644 --- a/src/intel/compiler/brw_fs.cpp +++ b/src/intel/compiler/brw_fs.cpp @@ -393,12 +393,6 @@ fs_inst::can_change_types() const !src[1].abs && !src[1].negate)); } -bool -fs_inst::has_side_effects() const -{ - return this->eot || backend_instruction::has_side_effects(); -} - void fs_reg::init() { -- cgit v1.2.3