Rather than acquiring the TerminatorInst and asking how many successors it has and comparing to 1 this patch just uses getSingleSuccessor() which should be equivalent.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Is this just a cleanup or is there any particular reason why you're doing this? I don't find it much more readable, FWIW.
Comment Actions
It was just cleanup. I was looking at the code and noticed that it was using getSinglePredecessor, but was taking a longer way to get the single successor.