[PATCH] D40098: [AMDGPU] Improve hazard checks for inline asm statements

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 16 19:37:46 PST 2017


arsenm added inline comments.


================
Comment at: lib/Target/AMDGPU/GCNHazardRecognizer.cpp:577
+    if (Op.isReg() && Op.isDef()) {
+      WaitStatesNeeded = std::max(WaitStatesNeeded, checkVALUHazardsHelper(Op, MRI));
+    }
----------------
Should this check every instruction type hazard?


https://reviews.llvm.org/D40098





More information about the llvm-commits mailing list