search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

handleMoveUp incorrectly re-uses valnos #42457

Open Quuxplusone opened 5 years ago

Quuxplusone commented 5 years ago
Bugzilla Link PR43487
Status NEW
Importance P enhancement
Reported by Matt Arsenault (Matthew.Arsenault@amd.com)
Reported on 2019-09-27 11:14:54 -0700
Last modified on 2019-09-27 11:14:54 -0700
Version trunk
Hardware PC All
CC llvm-bugs@lists.llvm.org, quentin.colombet@gmail.com, tpr.ll@botech.co.uk
Fixed by commit(s)
Attachments 0001-Add-wrong-valno-testcase.patch (1472 bytes, text/plain)
Blocks
Blocked by
See also 
Created attachment 22592
Patch adding testcase

While working on bug 43467 I found this testcase which produces incorrect
valnos from the path added in r326087

Assertion failed: (I->valno != std::next(I)->valno), function verify, file
../lib/CodeGen/LiveInterval.cpp, line 1071.

[16r,40r:1)[40r,64r:3)[64r,80r:3)[80r,80d:3)  0@64r 1@16r 2@80r 3@40r

valno 3 is used 3 times
Quuxplusone commented 5 years ago

Attached 0001-Add-wrong-valno-testcase.patch (1472 bytes, text/plain): Patch adding testcase