This is an archive of the discontinued LLVM Phabricator instance.

[SLP] Fixes crash in BoUpSLP::isGatherShuffledEntry()
ClosedPublic

Authored by vporpo on Feb 27 2023, 10:39 AM.

Diff Detail

Event Timeline

vporpo created this revision.Feb 27 2023, 10:39 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 27 2023, 10:39 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
vporpo requested review of this revision.Feb 27 2023, 10:39 AM
ABataev added inline comments.Feb 27 2023, 10:47 AM
llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
8471–8472

Better to replace TE->Scalars[I] with VL[I]

vporpo updated this revision to Diff 500850.Feb 27 2023, 10:55 AM

Replace TE->Scalars with VL, as suggested.

ABataev accepted this revision.Feb 27 2023, 10:56 AM

LG, thanks!

This revision is now accepted and ready to land.Feb 27 2023, 10:56 AM
This revision was landed with ongoing or failed builds.Feb 27 2023, 12:31 PM
This revision was automatically updated to reflect the committed changes.