Commit 3067c041 by Nicolas Capens Committed by Nicolas Capens

Eliminate unnecessary getDevice() calls.

BUG=18110152 Change-Id: Ib915aa01aa555e739d4c068ca845943057312fdb Reviewed-on: https://swiftshader-review.googlesource.com/1265Reviewed-by: 's avatarNicolas Capens <nicolascapens@google.com> Tested-by: 's avatarNicolas Capens <nicolascapens@google.com>
parent c78445cd
...@@ -40,6 +40,8 @@ Device *Context::device = 0; ...@@ -40,6 +40,8 @@ Device *Context::device = 0;
Context::Context(const egl::Config *config, const Context *shareContext) : mConfig(config) Context::Context(const egl::Config *config, const Context *shareContext) : mConfig(config)
{ {
device = getDevice();
setClearColor(0.0f, 0.0f, 0.0f, 0.0f); setClearColor(0.0f, 0.0f, 0.0f, 0.0f);
mState.depthClearValue = 1.0f; mState.depthClearValue = 1.0f;
...@@ -1293,7 +1295,6 @@ bool Context::isQueryParameterBool(GLenum pname) ...@@ -1293,7 +1295,6 @@ bool Context::isQueryParameterBool(GLenum pname)
// Applies the render target surface, depth stencil surface, viewport rectangle and scissor rectangle // Applies the render target surface, depth stencil surface, viewport rectangle and scissor rectangle
bool Context::applyRenderTarget() bool Context::applyRenderTarget()
{ {
Device *device = getDevice();
Framebuffer *framebuffer = getFramebuffer(); Framebuffer *framebuffer = getFramebuffer();
int width, height, samples; int width, height, samples;
...@@ -1342,7 +1343,6 @@ bool Context::applyRenderTarget() ...@@ -1342,7 +1343,6 @@ bool Context::applyRenderTarget()
// Applies the fixed-function state (culling, depth test, alpha blending, stenciling, etc) // Applies the fixed-function state (culling, depth test, alpha blending, stenciling, etc)
void Context::applyState(GLenum drawMode) void Context::applyState(GLenum drawMode)
{ {
Device *device = getDevice();
Framebuffer *framebuffer = getFramebuffer(); Framebuffer *framebuffer = getFramebuffer();
if(mState.cullFace) if(mState.cullFace)
...@@ -1526,9 +1526,7 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count) ...@@ -1526,9 +1526,7 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count)
{ {
return err; return err;
} }
Device *device = getDevice();
device->resetInputStreams(false); device->resetInputStreams(false);
for(int i = 0; i < MAX_VERTEX_ATTRIBS; i++) for(int i = 0; i < MAX_VERTEX_ATTRIBS; i++)
...@@ -1555,7 +1553,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count) ...@@ -1555,7 +1553,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count)
// Applies the indices and element array bindings // Applies the indices and element array bindings
GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode, GLenum type, TranslatedIndexData *indexInfo) GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode, GLenum type, TranslatedIndexData *indexInfo)
{ {
Device *device = getDevice();
GLenum err = mIndexDataManager->prepareIndexData(type, count, mState.elementArrayBuffer.get(), indices, indexInfo); GLenum err = mIndexDataManager->prepareIndexData(type, count, mState.elementArrayBuffer.get(), indices, indexInfo);
if(err == GL_NO_ERROR) if(err == GL_NO_ERROR)
...@@ -1568,8 +1565,6 @@ GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode ...@@ -1568,8 +1565,6 @@ GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode
void Context::applyTextures() void Context::applyTextures()
{ {
Device *device = getDevice();
for(int samplerIndex = 0; samplerIndex < MAX_TEXTURE_IMAGE_UNITS; samplerIndex++) for(int samplerIndex = 0; samplerIndex < MAX_TEXTURE_IMAGE_UNITS; samplerIndex++)
{ {
UNIMPLEMENTED(); UNIMPLEMENTED();
...@@ -1609,8 +1604,6 @@ void Context::applyTextures() ...@@ -1609,8 +1604,6 @@ void Context::applyTextures()
void Context::applyTexture(int index, Texture *baseTexture) void Context::applyTexture(int index, Texture *baseTexture)
{ {
Device *device = getDevice();
sw::Resource *resource = 0; sw::Resource *resource = 0;
if(baseTexture) if(baseTexture)
...@@ -1877,8 +1870,6 @@ void Context::clear(GLbitfield mask) ...@@ -1877,8 +1870,6 @@ void Context::clear(GLbitfield mask)
return; return;
} }
Device *device = getDevice();
unsigned int color = (unorm<8>(mState.colorClearValue.alpha) << 24) | unsigned int color = (unorm<8>(mState.colorClearValue.alpha) << 24) |
(unorm<8>(mState.colorClearValue.red) << 16) | (unorm<8>(mState.colorClearValue.red) << 16) |
(unorm<8>(mState.colorClearValue.green) << 8) | (unorm<8>(mState.colorClearValue.green) << 8) |
...@@ -1918,7 +1909,6 @@ void Context::clear(GLbitfield mask) ...@@ -1918,7 +1909,6 @@ void Context::clear(GLbitfield mask)
void Context::drawArrays(GLenum mode, GLint first, GLsizei count) void Context::drawArrays(GLenum mode, GLint first, GLsizei count)
{ {
Device *device = getDevice();
PrimitiveType primitiveType; PrimitiveType primitiveType;
int primitiveCount; int primitiveCount;
...@@ -1958,7 +1948,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void * ...@@ -1958,7 +1948,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void *
return error(GL_INVALID_OPERATION); return error(GL_INVALID_OPERATION);
} }
Device *device = getDevice();
PrimitiveType primitiveType; PrimitiveType primitiveType;
int primitiveCount; int primitiveCount;
...@@ -2001,8 +1990,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void * ...@@ -2001,8 +1990,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void *
void Context::finish() void Context::finish()
{ {
Device *device = getDevice();
device->finish(); device->finish();
} }
......
...@@ -43,6 +43,8 @@ Device *Context::device = 0; ...@@ -43,6 +43,8 @@ Device *Context::device = 0;
Context::Context(const egl::Config *config, const Context *shareContext) : mConfig(config) Context::Context(const egl::Config *config, const Context *shareContext) : mConfig(config)
{ {
device = getDevice();
mFenceHandleAllocator.setBaseHandle(0); mFenceHandleAllocator.setBaseHandle(0);
setClearColor(0.0f, 0.0f, 0.0f, 0.0f); setClearColor(0.0f, 0.0f, 0.0f, 0.0f);
...@@ -1670,7 +1672,6 @@ bool Context::getQueryParameterInfo(GLenum pname, GLenum *type, unsigned int *nu ...@@ -1670,7 +1672,6 @@ bool Context::getQueryParameterInfo(GLenum pname, GLenum *type, unsigned int *nu
// Applies the render target surface, depth stencil surface, viewport rectangle and scissor rectangle // Applies the render target surface, depth stencil surface, viewport rectangle and scissor rectangle
bool Context::applyRenderTarget() bool Context::applyRenderTarget()
{ {
Device *device = getDevice();
Framebuffer *framebuffer = getDrawFramebuffer(); Framebuffer *framebuffer = getDrawFramebuffer();
int width, height, samples; int width, height, samples;
...@@ -1729,7 +1730,6 @@ bool Context::applyRenderTarget() ...@@ -1729,7 +1730,6 @@ bool Context::applyRenderTarget()
// Applies the fixed-function state (culling, depth test, alpha blending, stenciling, etc) // Applies the fixed-function state (culling, depth test, alpha blending, stenciling, etc)
void Context::applyState(GLenum drawMode) void Context::applyState(GLenum drawMode)
{ {
Device *device = getDevice();
Framebuffer *framebuffer = getDrawFramebuffer(); Framebuffer *framebuffer = getDrawFramebuffer();
if(mState.cullFace) if(mState.cullFace)
...@@ -1949,7 +1949,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count) ...@@ -1949,7 +1949,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count)
return err; return err;
} }
Device *device = getDevice();
Program *program = getCurrentProgram(); Program *program = getCurrentProgram();
device->resetInputStreams(false); device->resetInputStreams(false);
...@@ -1984,7 +1983,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count) ...@@ -1984,7 +1983,6 @@ GLenum Context::applyVertexBuffer(GLint base, GLint first, GLsizei count)
// Applies the indices and element array bindings // Applies the indices and element array bindings
GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode, GLenum type, TranslatedIndexData *indexInfo) GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode, GLenum type, TranslatedIndexData *indexInfo)
{ {
Device *device = getDevice();
GLenum err = mIndexDataManager->prepareIndexData(type, count, mState.elementArrayBuffer.get(), indices, indexInfo); GLenum err = mIndexDataManager->prepareIndexData(type, count, mState.elementArrayBuffer.get(), indices, indexInfo);
if(err == GL_NO_ERROR) if(err == GL_NO_ERROR)
...@@ -1998,7 +1996,6 @@ GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode ...@@ -1998,7 +1996,6 @@ GLenum Context::applyIndexBuffer(const void *indices, GLsizei count, GLenum mode
// Applies the shaders and shader constants // Applies the shaders and shader constants
void Context::applyShaders() void Context::applyShaders()
{ {
Device *device = getDevice();
Program *programObject = getCurrentProgram(); Program *programObject = getCurrentProgram();
sw::VertexShader *vertexShader = programObject->getVertexShader(); sw::VertexShader *vertexShader = programObject->getVertexShader();
sw::PixelShader *pixelShader = programObject->getPixelShader(); sw::PixelShader *pixelShader = programObject->getPixelShader();
...@@ -2023,7 +2020,6 @@ void Context::applyTextures() ...@@ -2023,7 +2020,6 @@ void Context::applyTextures()
void Context::applyTextures(sw::SamplerType samplerType) void Context::applyTextures(sw::SamplerType samplerType)
{ {
Device *device = getDevice();
Program *programObject = getCurrentProgram(); Program *programObject = getCurrentProgram();
int samplerCount = (samplerType == sw::SAMPLER_PIXEL) ? MAX_TEXTURE_IMAGE_UNITS : MAX_VERTEX_TEXTURE_IMAGE_UNITS; // Range of samplers of given sampler type int samplerCount = (samplerType == sw::SAMPLER_PIXEL) ? MAX_TEXTURE_IMAGE_UNITS : MAX_VERTEX_TEXTURE_IMAGE_UNITS; // Range of samplers of given sampler type
...@@ -2075,7 +2071,6 @@ void Context::applyTextures(sw::SamplerType samplerType) ...@@ -2075,7 +2071,6 @@ void Context::applyTextures(sw::SamplerType samplerType)
void Context::applyTexture(sw::SamplerType type, int index, Texture *baseTexture) void Context::applyTexture(sw::SamplerType type, int index, Texture *baseTexture)
{ {
Device *device = getDevice();
Program *program = getCurrentProgram(); Program *program = getCurrentProgram();
int sampler = (type == sw::SAMPLER_PIXEL) ? index : 16 + index; int sampler = (type == sw::SAMPLER_PIXEL) ? index : 16 + index;
bool textureUsed = false; bool textureUsed = false;
...@@ -2379,9 +2374,7 @@ void Context::clear(GLbitfield mask) ...@@ -2379,9 +2374,7 @@ void Context::clear(GLbitfield mask)
{ {
return; return;
} }
Device *device = getDevice();
unsigned int color = (unorm<8>(mState.colorClearValue.alpha) << 24) | unsigned int color = (unorm<8>(mState.colorClearValue.alpha) << 24) |
(unorm<8>(mState.colorClearValue.red) << 16) | (unorm<8>(mState.colorClearValue.red) << 16) |
(unorm<8>(mState.colorClearValue.green) << 8) | (unorm<8>(mState.colorClearValue.green) << 8) |
...@@ -2426,7 +2419,6 @@ void Context::drawArrays(GLenum mode, GLint first, GLsizei count) ...@@ -2426,7 +2419,6 @@ void Context::drawArrays(GLenum mode, GLint first, GLsizei count)
return error(GL_INVALID_OPERATION); return error(GL_INVALID_OPERATION);
} }
Device *device = getDevice();
PrimitiveType primitiveType; PrimitiveType primitiveType;
int primitiveCount; int primitiveCount;
...@@ -2477,7 +2469,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void * ...@@ -2477,7 +2469,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void *
return error(GL_INVALID_OPERATION); return error(GL_INVALID_OPERATION);
} }
Device *device = getDevice();
PrimitiveType primitiveType; PrimitiveType primitiveType;
int primitiveCount; int primitiveCount;
...@@ -2526,8 +2517,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void * ...@@ -2526,8 +2517,6 @@ void Context::drawElements(GLenum mode, GLsizei count, GLenum type, const void *
void Context::finish() void Context::finish()
{ {
Device *device = getDevice();
device->finish(); device->finish();
} }
...@@ -2766,8 +2755,6 @@ void Context::blitFramebuffer(GLint srcX0, GLint srcY0, GLint srcX1, GLint srcY1 ...@@ -2766,8 +2755,6 @@ void Context::blitFramebuffer(GLint srcX0, GLint srcY0, GLint srcX1, GLint srcY1
GLint dstX0, GLint dstY0, GLint dstX1, GLint dstY1, GLint dstX0, GLint dstY0, GLint dstX1, GLint dstY1,
GLbitfield mask) GLbitfield mask)
{ {
Device *device = getDevice();
Framebuffer *readFramebuffer = getReadFramebuffer(); Framebuffer *readFramebuffer = getReadFramebuffer();
Framebuffer *drawFramebuffer = getDrawFramebuffer(); Framebuffer *drawFramebuffer = getDrawFramebuffer();
......
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