[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