Page MenuHomePhabricator

[SimpleLoopUnswitch] Don't consider unswitching `switch` insructions with one unique successor
ClosedPublic

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

Diff Detail

Repository
rL LLVM

Event Timeline

DaniilSuchkov created this revision.Jul 9 2019, 4:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 9 2019, 4:55 AM
skatkov accepted this revision.Jul 9 2019, 10:49 PM
skatkov added inline comments.
llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
2552 ↗(On Diff #208644)

Do you really need "== nullptr"?

This revision is now accepted and ready to land.Jul 9 2019, 10:49 PM

Addressed Serguei's comment. Serguei, could you please submit this patch? I don't have permissions to submit patches yet.

DaniilSuchkov edited the summary of this revision. (Show Details)Jul 9 2019, 11:17 PM
DaniilSuchkov marked an inline comment as done.Jul 9 2019, 11:35 PM
This revision was automatically updated to reflect the committed changes.