This little series adds value numbering support to the global code motion pass. All of the patches have my name on them but most of the credit goes to Connor and his instruction set. The instruction set was originally written for GVN but we merged it a while ago because we can also use it for regular CSE and it is much more efficient than the old list-based approach. Now that it's merged GVN is just a couple of lines.
I'm not going to recommend that we merge patch 3 yet. There are still some regressions with it and we should try and get that sorted out. That said, at some point, we just need to say "good enough", eat the costs, and enjoy the over-all benifit. That time may not yet have come though, so I won't push for turning it on. That said, I would like to merge the first two patches so they aren't floating around in my branch having to be rebased anymore. Jason Ekstrand (3): nir/gcm: Call nir_metadata_preserve nir/gcm: Add value numbering support i965/nir: Use GCM and GVN src/compiler/nir/nir.h | 2 +- src/compiler/nir/nir_opt_gcm.c | 32 +++++++++++++++++++++++++++----- src/mesa/drivers/dri/i965/brw_nir.c | 1 + 3 files changed, 29 insertions(+), 6 deletions(-) -- 2.5.0.400.gff86faf _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
