On Sat, May 19, 2012 at 11:59:24AM -0700, Jose Fonseca wrote:
>
>
> - Original Message -
> > +/* This method is the only one supported by gcc. Unions in
> > particular
> > + * are iffy, and read-through-converted-pointer is killed by strict
> > + * aliasing. OTOH, the compiler sees thro
- Original Message -
> This patch expects to be applied after the "Constants through
> builtins
> evaluation rewrite" for two reasons:
> - a line of that patch appears in the context
> - the ir_function side changes that the old method would need are not
> present (and they would confli
- Original Message -
> TGSI doesn't need an opcode, since registers are untyped (but beware
> once doubles come into the scene).
That's correct. This patch looks good.
Doubles will likely be handled similarly, with the difference that will. BTW, I
believe there were some patches for d
When considering which components of a variable were killed by an
assignment, constant propagation would previously just use the write
mask of the assignment. This worked if the LHS of the assignment was
simple, e.g.:
v.xy = ...; // (assign (xy) (var_ref v) ...)
But it did the wrong thing if the
Sounds good.
Jose
- Original Message -
> Fixes another case of sampler views being created by one context,
> shared by another, then deleted by the first, leaving a dangling
> pipe context pointer.
> ---
> src/mesa/state_tracker/st_context.c |2 +-
> 1 files changed, 1 insertions(+),
The IROUND implementation fix looks good.
Concerning this F_TO_I thought, I wonder if rounding to nearest was the true
intention below, and we just happened to be lucky so far. Anyway, easy to
revert to IROUND as needed, and not terribly important.
Jose
- Original Message -
> Use it w
On 05/18/2012 08:00 PM, Eric Anholt wrote:
The i965 driver needed this as well for hardware setup, so instead of
duplicating the logic, just save it off.
---
src/mesa/main/blend.c | 20
src/mesa/main/context.c | 24 +++-
src/mesa/main/mtypes.h |
Looks good.
Jose
- Original Message -
> The IROUND converted all arguments to 0 or 1. That's not what we
> wanted.
>
> NOTE: This is a candidate for the 8.0 branch.
> ---
> src/mesa/main/format_unpack.c |4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/s
