Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-29 Thread Jason Ekstrand
On Wed, Aug 29, 2018 at 3:19 PM Caio Marcelo de Oliveira Filho < caio.olive...@intel.com> wrote: > Jason Ekstrand writes: > > >> >> +static bool > >> >> +remove_dead_write_vars_local(struct state *state, nir_block *block) > >> >> +{ > >> >> + bool progress = false; > >> >> + > >> >> + struct

Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-29 Thread Caio Marcelo de Oliveira Filho
Jason Ekstrand writes: >> >> +static bool >> >> +remove_dead_write_vars_local(struct state *state, nir_block *block) >> >> +{ >> >> + bool progress = false; >> >> + >> >> + struct util_dynarray unused_writes; >> >> + util_dynarray_init(&unused_writes, state->mem_ctx); >> >> + >> >> + nir_

Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-28 Thread Jason Ekstrand
On Mon, Aug 27, 2018 at 4:55 PM Caio Marcelo de Oliveira Filho < caio.olive...@intel.com> wrote: > (Disregard the incomplete mail, still adapting to notmuch-emacs). > > Jason Ekstrand writes: > > >> +static nir_deref_path * > >> +get_path(struct state *state, nir_deref_instr *deref) > >> +{ > >>

Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-27 Thread Caio Marcelo de Oliveira Filho
(Disregard the incomplete mail, still adapting to notmuch-emacs). Jason Ekstrand writes: >> +static nir_deref_path * >> +get_path(struct state *state, nir_deref_instr *deref) >> +{ >> + struct hash_entry *entry = _mesa_hash_table_search(state->paths, >> deref); >> + if (!entry) { >> + n

Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-27 Thread Caio Marcelo de Oliveira Filho
Jason Ekstrand writes: > On Wed, Aug 15, 2018 at 4:57 PM Caio Marcelo de Oliveira Filho < > caio.olive...@intel.com> wrote: > >> Instead of doing this as part of the existing (local) copy prop vars >> pass. This is an intermediate step before changing both the dead >> write and the copy prop var

Re: [Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-22 Thread Jason Ekstrand
On Wed, Aug 15, 2018 at 4:57 PM Caio Marcelo de Oliveira Filho < caio.olive...@intel.com> wrote: > Instead of doing this as part of the existing (local) copy prop vars > pass. This is an intermediate step before changing both the dead > write and the copy prop vars to act on the whole program ins

[Mesa-dev] [PATCH 5/9] nir: Add a local dead write vars removal pass

2018-08-15 Thread Caio Marcelo de Oliveira Filho
Instead of doing this as part of the existing (local) copy prop vars pass. This is an intermediate step before changing both the dead write and the copy prop vars to act on the whole program instead of on local blocks. The nature of data we store and the way we iterate is different enough that wo