[Mesa-dev] [PATCH 04/15] i965/fs: Don't emit null MOVs in CSE.

2013-10-29 Thread Matt Turner
We'd like to CSE some instructions, like CMP, that often have null destinations. Instead of replacing them with MOVs to null, just don't emit the MOV. --- Updated version: Replaced two comments with Eric's suggested comment. Left the comment about the basic block boundary in place. Let me know if

Re: [Mesa-dev] [PATCH 04/15] i965/fs: Don't emit null MOVs in CSE.

2013-10-28 Thread Eric Anholt
Matt Turner writes: > We'd like to CSE some instructions, like CMP, that often have null > destinations. Instead of replacing them with MOVs to null, just don't > emit the MOV. > --- > src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 42 > +++- > 1 file changed, 25 inserti

[Mesa-dev] [PATCH 04/15] i965/fs: Don't emit null MOVs in CSE.

2013-10-28 Thread Matt Turner
We'd like to CSE some instructions, like CMP, that often have null destinations. Instead of replacing them with MOVs to null, just don't emit the MOV. --- src/mesa/drivers/dri/i965/brw_fs_cse.cpp | 42 +++- 1 file changed, 25 insertions(+), 17 deletions(-) diff --git a