[llvm] [AMDGPU] Add flag to force emit s_nop (PR #117839)

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 27 06:06:09 PST 2024


================
@@ -325,7 +330,9 @@ unsigned GCNHazardRecognizer::PreEmitNoops(MachineInstr *MI) {
   unsigned W = PreEmitNoopsCommon(MI);
   fixHazards(MI);
   CurrCycleInstr = nullptr;
-  return W;
+  unsigned NopPad =
+      NopPadding.getNumOccurrences() && !MI->isTerminator() ? NopPadding : 0;
----------------
jayfoad wrote:

Because of a rule that all terminators have to be consecutive at the end of the block?

You could still have a nop before the first terminator.

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


More information about the llvm-commits mailing list