This is an archive of the discontinued LLVM Phabricator instance.

[Hexagon] Remove unintended fallthrough from MC duplex code
ClosedPublic

Authored by rnk on Nov 1 2018, 12:41 PM.

Details

Summary

I recently added these fall through annotations, but it's not clear if
the code is intended to fall through. In this patch, I changed them to
breaks, which seems more likely to be correct. If so, please approve
this patch and I will commit it. If not, I will reupload this patch to
remove the FIXME comments from the annotations.

Event Timeline

rnk created this revision.Nov 1 2018, 12:41 PM
kparzysz accepted this revision.Nov 1 2018, 12:45 PM

Thanks!

This revision is now accepted and ready to land.Nov 1 2018, 12:45 PM
This revision was automatically updated to reflect the committed changes.