[PATCH] D29048: [RegisterCoalescer] Do not call LiveIntervals::getInstructionIndex with a DBG_VALUE

Brendon Cahoon via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 26 15:57:17 PST 2017


bcahoon updated this revision to Diff 85981.
bcahoon added a comment.
Herald added a reviewer: tstellarAMD.
Herald added a subscriber: nhaehnle.

I've created a test case using the AMDGPU target that shows the error.  This required a couple of steps. I took an existing AMDGPU lit test, and added debug metadata manually.  Then, created MIR for the test by stopping after the TwoAddressInstruction pass.  Then, I needed to insert a DBG_VALUE instruction in the correct location. I'm not really sure if it's possible for a DBG_VALUE instruction to appear in the specific place that I put it. But, with these changes, the test case causes an assert because there is a call to getInstructionIndex with a DBG_VALUE insruction.


https://reviews.llvm.org/D29048

Files:
  lib/CodeGen/RegisterCoalescer.cpp
  test/CodeGen/AMDGPU/regcoalesce-dbg.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29048.85981.patch
Type: text/x-patch
Size: 6136 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170126/cc002fd7/attachment.bin>


More information about the llvm-commits mailing list