[llvm-bugs] [Bug 44139] combineInsertEltToShuffle - ASSERT: idx < size()

[via llvm-bugs]

https://bugs.llvm.org/show_bug.cgi?id=44139

Sanjay Patel <spatel+llvm at rotateright.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
 Fixed By Commit(s)|                            |214683f3b2d6
             Status|NEW                         |RESOLVED

--- Comment #3 from Sanjay Patel <spatel+llvm at rotateright.com> ---
(In reply to Simon Pilgrim from comment #2)
> I'm guessing DAGCombine is missing out of bounds insert/extractelement
> handling compared to instsimplify:
> 
> Opt codegen: https://gcc.godbolt.org/z/uDSBW8

Yep - I thought I might've created the loophole that allowed the crash, but it
exists independently of:
https://reviews.llvm.org/rG85a2146c1559

Always surprised at the order that we can perform combines vs. creating
nodes...
so I might've introduced a different bug by deleting a clause from DAGCombiner
in that patch.

Will try to keep an eye on fuzzer mail.

Should be fixed with:
https://reviews.llvm.org/rG214683f3b2d6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20191125/b3c0129b/attachment.html>