This is an archive of the discontinued LLVM Phabricator instance.

Define BBEntry::hasIndirectBranch. NFC
ClosedPublic

Authored by rahmanl on Aug 21 2023, 8:32 AM.

Diff Detail

Event Timeline

rahmanl created this revision.Aug 21 2023, 8:32 AM
Herald added a project: Restricted Project. · View Herald Transcript
rahmanl requested review of this revision.Aug 21 2023, 8:32 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2023, 8:32 AM
rahmanl updated this revision to Diff 552035.Aug 21 2023, 8:34 AM

Remove .gitignore from the patch.

This revision was not accepted when it landed; it landed in state Needs Review.Aug 21 2023, 8:39 AM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.

Looks good (should it have had a unit test?). Not sure why this was landed without review though?

Looks good (should it have had a unit test?). Not sure why this was landed without review though?

The field HasIndirectBranch was already added and unit-tests already covered it. This patch just added a getter for that field.