[PATCH] D83101: [Scalarizer] ExtractElement handling w/ constant extract index

Tue Jul 7 07:42:05 PDT 2020

bjope added a comment.

In D83101#2136116 <https://reviews.llvm.org/D83101#2136116>, @lebedev.ri wrote:

> > Ah, maybe I was so occupied reducing the fault so I missed that there was another fix.  I actually did take a look in github just to avoid reporting a problem that had been fixed, but must have done it just before rG16266e63963ad6ee27ad21983a9366ab313dfd03 <https://reviews.llvm.org/rG16266e63963ad6ee27ad21983a9366ab313dfd03> landed.
>
> Nono, rG16266e63963ad6ee27ad21983a9366ab313dfd03 <https://reviews.llvm.org/rG16266e63963ad6ee27ad21983a9366ab313dfd03> is the fix as a reaction to the bug you reported, your report wasn't a duplicate, sorry for confusion.

Ah, thanks! (That was such a quick fix/response so I thought someone else had discovered the same problem.)

>> I'll fetch, rebuild, and test. Let you know if it didn't help.
> 
> Yes, please, thank you!

The fix works fine. So I don't know about more problems right now.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D83101/new/

https://reviews.llvm.org/D83101