[llvm] [AMDGPU] Mark S_NOP as having side effects (PR #65745)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 8 05:40:13 PDT 2023


================
@@ -72,8 +72,7 @@
   ; GFX908-NEXT:    ; implicit-def: $vgpr59
   ; GFX908-NEXT:    ; implicit-def: $vgpr60
   ; GFX908-NEXT:    ; implicit-def: $vgpr61
-  ; GFX908-NEXT:    s_nop 0
-  ; GFX908-NEXT:    s_nop 0
+  ; GFX908-NEXT:    s_nop 1
----------------
jayfoad wrote:

GCNHazardRecognizer wants to delay for 2 cycles here. If there was already an S_NOP 0 here it would add another one, instead of modifying it to be an S_NOP 1.

https://github.com/llvm/llvm-project/pull/65745


More information about the llvm-commits mailing list