glsl/types: Rename sampler_type to sampled_type
It's a bit more descriptive since it is the base type that you get when you sample from it. Also, the next commit adds a bare "sampler" type and we need glsl_type::sampler_type available for a public static member. Reviewed-by: Jordan Justen <jordan.l.justen@intel.com>
This commit is contained in:
@@ -2267,7 +2267,7 @@ get_type_name_for_precision_qualifier(const glsl_type *type)
|
||||
type->sampler_array + 2 * type->sampler_shadow;
|
||||
const unsigned offset = type->base_type == GLSL_TYPE_SAMPLER ? 0 : 4;
|
||||
assert(type_idx < 4);
|
||||
switch (type->sampler_type) {
|
||||
switch (type->sampled_type) {
|
||||
case GLSL_TYPE_FLOAT:
|
||||
switch (type->sampler_dimensionality) {
|
||||
case GLSL_SAMPLER_DIM_1D: {
|
||||
@@ -2953,7 +2953,7 @@ apply_image_qualifier_to_variable(const struct ast_type_qualifier *qual,
|
||||
"used on image function parameters");
|
||||
}
|
||||
|
||||
if (qual->image_base_type != base_type->sampler_type) {
|
||||
if (qual->image_base_type != base_type->sampled_type) {
|
||||
_mesa_glsl_error(loc, state, "format qualifier doesn't match the "
|
||||
"base data type of the image");
|
||||
}
|
||||
|
@@ -2921,7 +2921,7 @@ builtin_builder::add_image_function(const char *name,
|
||||
ir_function *f = new(mem_ctx) ir_function(name);
|
||||
|
||||
for (unsigned i = 0; i < ARRAY_SIZE(types); ++i) {
|
||||
if ((types[i]->sampler_type != GLSL_TYPE_FLOAT ||
|
||||
if ((types[i]->sampled_type != GLSL_TYPE_FLOAT ||
|
||||
(flags & IMAGE_FUNCTION_SUPPORTS_FLOAT_DATA_TYPE)) &&
|
||||
(types[i]->sampler_dimensionality == GLSL_SAMPLER_DIM_MS ||
|
||||
!(flags & IMAGE_FUNCTION_MS_ONLY)))
|
||||
@@ -5238,7 +5238,7 @@ builtin_builder::_image_prototype(const glsl_type *image_type,
|
||||
unsigned flags)
|
||||
{
|
||||
const glsl_type *data_type = glsl_type::get_instance(
|
||||
image_type->sampler_type,
|
||||
image_type->sampled_type,
|
||||
(flags & IMAGE_FUNCTION_HAS_VECTOR_DATA_TYPE ? 4 : 1),
|
||||
1);
|
||||
const glsl_type *ret_type = (flags & IMAGE_FUNCTION_RETURNS_VOID ?
|
||||
|
@@ -1442,7 +1442,7 @@ ir_texture::set_sampler(ir_dereference *sampler, const glsl_type *type)
|
||||
assert(sampler->type->base_type == GLSL_TYPE_SAMPLER);
|
||||
assert(sampler->type->sampler_dimensionality == GLSL_SAMPLER_DIM_MS);
|
||||
} else {
|
||||
assert(sampler->type->sampler_type == (int) type->base_type);
|
||||
assert(sampler->type->sampled_type == (int) type->base_type);
|
||||
if (sampler->type->sampler_shadow)
|
||||
assert(type->vector_elements == 4 || type->vector_elements == 1);
|
||||
else
|
||||
|
@@ -43,7 +43,7 @@ TEST(sampler_types, TYPE) \
|
||||
const glsl_type *type = glsl_type::TYPE##_type; \
|
||||
EXPECT_EQ(GLSL_TYPE_SAMPLER, type->base_type); \
|
||||
EXPECT_EQ(DIM, type->sampler_dimensionality); \
|
||||
EXPECT_EQ(DATA_TYPE, type->sampler_type); \
|
||||
EXPECT_EQ(DATA_TYPE, type->sampled_type); \
|
||||
ARR; \
|
||||
SHAD; \
|
||||
EXPECT_EQ(COMPS, type->coordinate_components()); \
|
||||
|
@@ -50,7 +50,7 @@ glsl_type::glsl_type(GLenum gl_type,
|
||||
gl_type(gl_type),
|
||||
base_type(base_type),
|
||||
sampler_dimensionality(0), sampler_shadow(0), sampler_array(0),
|
||||
sampler_type(0), interface_packing(0),
|
||||
sampled_type(0), interface_packing(0),
|
||||
vector_elements(vector_elements), matrix_columns(matrix_columns),
|
||||
length(0)
|
||||
{
|
||||
@@ -74,7 +74,7 @@ glsl_type::glsl_type(GLenum gl_type, glsl_base_type base_type,
|
||||
gl_type(gl_type),
|
||||
base_type(base_type),
|
||||
sampler_dimensionality(dim), sampler_shadow(shadow),
|
||||
sampler_array(array), sampler_type(type), interface_packing(0),
|
||||
sampler_array(array), sampled_type(type), interface_packing(0),
|
||||
length(0)
|
||||
{
|
||||
mtx_lock(&glsl_type::mutex);
|
||||
@@ -100,7 +100,7 @@ glsl_type::glsl_type(const glsl_struct_field *fields, unsigned num_fields,
|
||||
gl_type(0),
|
||||
base_type(GLSL_TYPE_STRUCT),
|
||||
sampler_dimensionality(0), sampler_shadow(0), sampler_array(0),
|
||||
sampler_type(0), interface_packing(0),
|
||||
sampled_type(0), interface_packing(0),
|
||||
vector_elements(0), matrix_columns(0),
|
||||
length(num_fields)
|
||||
{
|
||||
@@ -140,7 +140,7 @@ glsl_type::glsl_type(const glsl_struct_field *fields, unsigned num_fields,
|
||||
gl_type(0),
|
||||
base_type(GLSL_TYPE_INTERFACE),
|
||||
sampler_dimensionality(0), sampler_shadow(0), sampler_array(0),
|
||||
sampler_type(0), interface_packing((unsigned) packing),
|
||||
sampled_type(0), interface_packing((unsigned) packing),
|
||||
vector_elements(0), matrix_columns(0),
|
||||
length(num_fields)
|
||||
{
|
||||
@@ -178,7 +178,7 @@ glsl_type::glsl_type(const char *subroutine_name) :
|
||||
gl_type(0),
|
||||
base_type(GLSL_TYPE_SUBROUTINE),
|
||||
sampler_dimensionality(0), sampler_shadow(0), sampler_array(0),
|
||||
sampler_type(0), interface_packing(0),
|
||||
sampled_type(0), interface_packing(0),
|
||||
vector_elements(1), matrix_columns(1),
|
||||
length(0)
|
||||
{
|
||||
@@ -394,7 +394,7 @@ _mesa_glsl_release_types(void)
|
||||
glsl_type::glsl_type(const glsl_type *array, unsigned length) :
|
||||
base_type(GLSL_TYPE_ARRAY),
|
||||
sampler_dimensionality(0), sampler_shadow(0), sampler_array(0),
|
||||
sampler_type(0), interface_packing(0),
|
||||
sampled_type(0), interface_packing(0),
|
||||
vector_elements(0), matrix_columns(0),
|
||||
length(length), name(NULL)
|
||||
{
|
||||
|
@@ -121,7 +121,7 @@ struct glsl_type {
|
||||
unsigned sampler_dimensionality:3; /**< \see glsl_sampler_dim */
|
||||
unsigned sampler_shadow:1;
|
||||
unsigned sampler_array:1;
|
||||
unsigned sampler_type:2; /**< Type of data returned using this
|
||||
unsigned sampled_type:2; /**< Type of data returned using this
|
||||
* sampler or image. Only \c
|
||||
* GLSL_TYPE_FLOAT, \c GLSL_TYPE_INT,
|
||||
* and \c GLSL_TYPE_UINT are valid.
|
||||
|
@@ -1235,7 +1235,7 @@ fs_visitor::emit_percomp(const fs_builder &bld, const fs_inst &inst,
|
||||
static brw_reg_type
|
||||
get_image_base_type(const glsl_type *type)
|
||||
{
|
||||
switch ((glsl_base_type)type->sampler_type) {
|
||||
switch ((glsl_base_type)type->sampled_type) {
|
||||
case GLSL_TYPE_UINT:
|
||||
return BRW_REGISTER_TYPE_UD;
|
||||
case GLSL_TYPE_INT:
|
||||
|
Reference in New Issue
Block a user