[PATCH] D51849: [RegisterCoalescer] Avoid "Use not jointly dominated by defs" in removePartialRedundancy

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 10 00:42:56 PDT 2018


tpr created this revision.
Herald added subscribers: llvm-commits, nhaehnle, jvesely, qcolombet, MatzeB.

removePartialRedundancy was using extendToIndices on each subrange
without passing in an Undefs vector containing main reg defs that are
undef in the subrange, causing the above assert.

Unfortunately I can only reproduce this in an ll test. Turning it into a
mir test makes the problem go away.

Change-Id: I4f3ba2afb20d79f9467afb3d882f0328d38531be


Repository:
  rL LLVM

https://reviews.llvm.org/D51849

Files:
  lib/CodeGen/RegisterCoalescer.cpp
  test/CodeGen/AMDGPU/regcoal-removepartial-redundancy-not-jointly-dominated.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51849.164622.patch
Type: text/x-patch
Size: 13146 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180910/5667c47c/attachment.bin>


More information about the llvm-commits mailing list