Committed in r183237.
On Tue, Jun 4, 2013 at 9:32 AM, Jordan Rose <[email protected]> wrote: > Cute, good catch. I think you should just break up that monster > conditional into a few ifs, or just |= the enum case into > switchExclusivelyCovered, but semantically this looks right. > > Jordan > > > On Jun 4, 2013, at 0:42 , David Majnemer <[email protected]> wrote: > > > Hi, > > > > clang does not consider the case where a SwitchStmt satisfied > isAllEnumCasesCovered as well as having no cases at all (i.e. the enum it > covers has no enumerators). > > > > The attached patch adds a successor in this case which gives us a proper > CFG. > > > > This fixes PR16212. > > > > -- > > David Majnemer > > > _______________________________________________ > > cfe-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > > >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
