Fix eliminating stores to pointer arguments.

If a store address has no definition, it must be a pointer argument. Unlike stores to alloca's, these are not dead. Change-Id: I66e50b14ffea1cfc61fc756b5545ab8ae28b4bf8 Reviewed-on: https://swiftshader-review.googlesource.com/8573Tested-by: Nicolas Capens <capn@google.com> Reviewed-by: Alexis Hétu <sugoi@google.com> Reviewed-by: Nicolas Capens <capn@google.com>

Fix eliminating stores to pointer arguments.
da721421 · Nicolas Capens · 39061652 · da721421
Commit da721421 authored Jan 27, 2017 by Nicolas Capens
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

Optimizer.cpp src/Reactor/Optimizer.cpp +1 -1

No files found.
--- a/src/Reactor/Optimizer.cpp
+++ b/src/Reactor/Optimizer.cpp
@@ -454,7 +454,7 @@ namespace
 			{
 				Ice::Inst *def = definition[address];
-				if(!def || llvm::isa<Ice::InstAlloca>(def))
+				if(def && llvm::isa<Ice::InstAlloca>(def))
 				{
 					return uses[address].size() == 1;   // Dead if this store is the only use
 				}