Skip to content

S
swiftshader
Commit 44be0942 by Ben Clayton
Options

Pipeline: Fix return type of SIMD::Pointer::limit()

`limit()` is the same for all lanes, so use `rr::Int`, not `SIMD::Int`. This was using an unqualified scope, so likely just a screwup when I first wrote this. Bug: None. A simple fix. Change-Id: I0df67fb88f9973dd522a5555888173ba1f5ae594 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/48689Reviewed-by: 's avatarAlexis Hétu <sugoi@google.com> Tested-by: 's avatarBen Clayton <bclayton@google.com>
parent 200a4633
Showing with 2 additions and 2 deletions
src/Pipeline/ShaderCore.cpp
...@@ -1083,7 +1083,7 @@ bool Pointer::isStaticallyInBounds(unsigned int accessSize, OutOfBoundsBehavior ...@@ -1083,7 +1083,7 @@ bool Pointer::isStaticallyInBounds(unsigned int accessSize, OutOfBoundsBehavior
return true; return true;
} }
Int Pointer::limit() const rr::Int Pointer::limit() const
{ {
return dynamicLimit + staticLimit; return dynamicLimit + staticLimit;
} }
......
src/Pipeline/ShaderCore.hpp
...@@ -121,7 +121,7 @@ struct Pointer ...@@ -121,7 +121,7 @@ struct Pointer
bool isStaticallyInBounds(unsigned int accessSize, OutOfBoundsBehavior robustness) const; bool isStaticallyInBounds(unsigned int accessSize, OutOfBoundsBehavior robustness) const;
Int limit() const; rr::Int limit() const;
// Returns true if all offsets are sequential // Returns true if all offsets are sequential
// (N+0*step, N+1*step, N+2*step, N+3*step) // (N+0*step, N+1*step, N+2*step, N+3*step)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment