[PATCH] D119399: [MachineSink] Allow target to adjust sink insertion point

Carl Ritson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 9 19:05:38 PST 2022


critson added a comment.

The test is based on new wave transform control flow, and does not occur with current control flow which uses pseudo instructions which are typically lowered after machine sinking.



================
Comment at: llvm/test/CodeGen/AMDGPU/sink-after-control-flow.mir:71
+  ; GFX10-NEXT:   $exec_lo = S_OR_B32 $exec_lo, [[S_XOR_B32_5]], implicit-def $scc
+  ; GFX10-NEXT:   [[V_LSHRREV_B32_e64_:%[0-9]+]]:vgpr_32 = V_LSHRREV_B32_e64 [[S_MOV_B32_3]], [[DEF1]], implicit $exec
+  ; GFX10-NEXT:   [[S_MOV_B32_5:%[0-9]+]]:sreg_32 = S_MOV_B32 31
----------------
Without this change this instruction is inserted before the S_OR_B32 above.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119399/new/

https://reviews.llvm.org/D119399



More information about the llvm-commits mailing list