[PATCH] D18549: AMDGPU/SI: Add MachineBasicBlock parameter to SIInstrInfo::insertWaitStates

Tom Stellard via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 07:37:39 PDT 2016


tstellarAMD added a comment.

In http://reviews.llvm.org/D18549#385699, @arsenm wrote:

> LGTM. Is this ever actually different than MI->getParent()?


MI is an iterator, so it could be BasicBlock::end(), which isn't always an instruction and doesn't have a parent.


http://reviews.llvm.org/D18549





More information about the llvm-commits mailing list