[PATCH] D29452: AMDGPU/SI: DAGMutation for removing deps between CSEs

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 10 10:56:34 PST 2017


arsenm added a comment.

Should this try to delete the redundant moves itself? MachineCSE runs earlier on SSA, so this isn't getting cleaned up. It helps with the clustering, but I get sequences of re-initing m0 to -1 like:
	s_mov_b32 m0, -1
	s_mov_b32 m0, -1
	s_mov_b32 m0, -1
	s_mov_b32 m0, -1
	v_mov_b32_e32 v50, 0x4a8
	ds_read2_b32 v[82:83], v50 offset1:1
	ds_read2_b32 v[50:51], v35 offset1:1


https://reviews.llvm.org/D29452





More information about the llvm-commits mailing list