This is an archive of the discontinued LLVM Phabricator instance.

[SimpleLoopUnswitch] Add a test case exposing a bug
ClosedPublic

Authored by DaniilSuchkov on Jul 9 2019, 4:50 AM.

Details

Summary

This test exposes a bug in SimpleLoopUnswitch that leads to a crash on assert(SuccessorsCount > 1 && "Cannot unswitch a condition without multiple distinct successors!"); when SimpleLoopUnswitch considers unswitching of a loop by a switch with one successor.

Fix will be submitted soon.

Diff Detail

Repository
rL LLVM

Event Timeline

DaniilSuchkov created this revision.Jul 9 2019, 4:50 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 4:50 AM
skatkov accepted this revision.Jul 9 2019, 10:44 PM
This revision is now accepted and ready to land.Jul 9 2019, 10:44 PM
DaniilSuchkov added a comment.EditedJul 9 2019, 10:47 PM

@skatkov, could you please submit this patch? I don't have permissions to submit patches yet.

This revision was automatically updated to reflect the committed changes.