[PATCH] D34849: [AMDGPU] Add missing hazard for DPP-after-EXEC-write
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 10 15:14:38 PDT 2017
arsenm added inline comments.
================
Comment at: test/CodeGen/AMDGPU/inserted-wait-states.mir:495
+# VI: S_MOV_B64
+# VI: S_NOP
+# VI: S_NOP
----------------
Should check the NOP argument is 0. Ideally these would be combined into one nop with the right value later. Also why not use VI-NEXT for all of them?
https://reviews.llvm.org/D34849
More information about the llvm-commits
mailing list