[PATCH v3 3/3] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon

[PATCH v3 2/3] Replace int2float() with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 51 1 file changed, 28 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

[PATCH v3 1/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 52 +++--- drivers/gpu/drm/radeon/r600_blit_shaders.h |1 + 3 files changed, 40 insertions(+), 79 deletions

[PATCH v3 3/3] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon

[PATCH v3 2/3] Replace int2float() with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 51 1 file changed, 28 insertions(+), 23 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

[PATCH v3 1/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 52 +++--- drivers/gpu/drm/radeon/r600_blit_shaders.h |1 + 3 files changed, 40 insertions(+), 79 deletions

[Patch v2 4/4] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +- 2 files changed,

[Patch v2 3/4] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst Conflicts: drivers/gpu/drm/radeon/r600_blit_kms.c --- drivers/gpu/drm/radeon/r600_blit.c | 67 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 43 +++--- drivers/gpu/drm/radeon/r600_blit_shaders.h

[Patch v2 2/4] Replace i2f() in r600_blit_kms.c with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 51 +--- 1 file changed, 21 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit_kms.c b/drivers/gpu

[Patch v2 1/4] Replace i2f() in r600_blit.c with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 50 1 file changed, 28 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

[Patch v2 4/4] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +- 2 files changed,

[Patch v2 3/4] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst Conflicts: drivers/gpu/drm/radeon/r600_blit_kms.c --- drivers/gpu/drm/radeon/r600_blit.c | 67 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 43 +++--- drivers/gpu/drm/radeon/r600_blit_shaders.h

[Patch v2 2/4] Replace i2f() in r600_blit_kms.c with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 51 +--- 1 file changed, 21 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit_kms.c b/drivers/gpu

[Patch v2 1/4] Replace i2f() in r600_blit.c with an optimized version.

. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 50 1 file changed, 28 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

[PATCH 4/3] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +- 2 files changed,

[[PATCH][RESENT] 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 45 +++ drivers/gpu/drm/radeon/r600_blit_shaders.h |1 + 3 files changed, 40 insertions(+), 72 deletions

[[PATCH][RESENT] 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 53 ++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit_kms.c b/drivers/gpu/drm

[[PATCH][RESENT] 1/3] Replace i2f() in r600_blit.c with an optimized version.

that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 52 +--- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

Re: [PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

y avoided. Steven Fuerst ___ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH 4/3] Annotate int2float() as being a pure function.

This allows gcc to fold duplicate calls into a single call. Since the current users do actually call it multiple times with the same arguments, this is an obvious win. --- drivers/gpu/drm/radeon/r600_blit.c |2 +- drivers/gpu/drm/radeon/r600_blit_shaders.h |2 +- 2 files changed,

[[PATCH][RESENT] 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 45 +++ drivers/gpu/drm/radeon/r600_blit_shaders.h |1 + 3 files changed, 40 insertions(+), 72 deletions

[[PATCH][RESENT] 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 53 ++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit_kms.c b/drivers/gpu/drm

[[PATCH][RESENT] 1/3] Replace i2f() in r600_blit.c with an optimized version.

that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 52 +--- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600_blit.c b/drivers/gpu/drm

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

y avoided. Steven Fuerst

[PATCH 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

inside general drm infrastructure. However, at the moment only the radeon driver uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 45 +++ drivers/gpu/drm/radeon

PATCH 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

full 32 bits. The routine is now exact up to 2^24. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 53 ++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

32 bits. The routine is now exact up to 2^24. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 52 +--- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a

[PATCH 3/3] Rename i2f() to int2float(), and make it global so one copy can be removed.

inside general drm infrastructure. However, at the moment only the radeon driver uses it. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 66 ++-- drivers/gpu/drm/radeon/r600_blit_kms.c | 45 +++ drivers/gpu/drm/radeon

PATCH 2/3] Replace i2f() in r600_blit_kms.c with an optimized version.

full 32 bits. The routine is now exact up to 2^24. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit_kms.c | 53 ++-- 1 file changed, 23 insertions(+), 30 deletions(-) diff --git

[PATCH 1/3] Replace i2f() in r600_blit.c with an optimized version.

32 bits. The routine is now exact up to 2^24. Above that, we truncate which is equivalent to rounding towards zero. Signed-off-by: Steven Fuerst --- drivers/gpu/drm/radeon/r600_blit.c | 52 +--- 1 file changed, 30 insertions(+), 22 deletions(-) diff --git a

Optimize i2f()

This means we don't * need to check to see if we are above 2^24 or not. */ fraction = ror32(x, msb - I2F_FRAC_BITS) & I2F_MASK; exponent = (127 + msb) << I2F_FRAC_BITS; return fraction + exponent; } Steven Fuerst ___ dri-devel m

Optimize i2f()

This means we don't * need to check to see if we are above 2^24 or not. */ fraction = ror32(x, msb - I2F_FRAC_BITS) & I2F_MASK; exponent = (127 + msb) << I2F_FRAC_BITS; return fraction + exponent; } Steven Fuerst -- next part -- An HTML attachment was sc