This is an archive of the discontinued LLVM Phabricator instance.

[InstCombine] Update InstCombine to use poison instead of undef for shufflevector's placeholder (3/3)
ClosedPublic

Authored by hyeongyukim on Sep 22 2021, 4:05 AM.

Details

Summary

This patch is for fixing potential shufflevector-related bugs like D93818.
As D93818, this patch change shufflevector's default placeholder to poison.
To reduce risk, it was divided into several patches, and this patch is for InstCombineVectorOps.

Diff Detail

Event Timeline

hyeongyukim created this revision.Sep 22 2021, 4:05 AM
hyeongyukim requested review of this revision.Sep 22 2021, 4:05 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2021, 4:05 AM
hyeongyukim retitled this revision from [InstCombine] Update InstCombine to use poison instead of undef for shufflevector's placeholder (2/3) to [InstCombine] Update InstCombine to use poison instead of undef for shufflevector's placeholder (3/3).Sep 22 2021, 4:07 AM

reformat code

spatel accepted this revision.Sep 22 2021, 5:54 AM

LGTM

This revision is now accepted and ready to land.Sep 22 2021, 5:54 AM
This revision was landed with ongoing or failed builds.Sep 22 2021, 8:48 AM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptSep 22 2021, 8:48 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript