diff options
| author | Kenneth Graunke <[email protected]> | 2017-10-18 11:22:43 -0700 |
|---|---|---|
| committer | Kenneth Graunke <[email protected]> | 2017-10-19 10:19:20 -0700 |
| commit | 3d112a7cd49656da3dde5f9b476b6ddc2d984528 (patch) | |
| tree | c92d96c573f087a06a18f706ded0d5a1d34afe1b /src/intel/compiler/brw_fs.cpp | |
| parent | 77b8392858815625ee7909cf9e866043dab9d074 (diff) | |
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 <[email protected]>
Tested-by: Pallavi G <[email protected]>
Diffstat (limited to 'src/intel/compiler/brw_fs.cpp')
| -rw-r--r-- | src/intel/compiler/brw_fs.cpp | 6 |
1 files changed, 0 insertions, 6 deletions
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() { |